Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

llvm-wrapper: remove some unused headers #117676

Closed
wants to merge 4 commits into from
Closed

llvm-wrapper: remove some unused headers #117676

wants to merge 4 commits into from

Commits on Nov 7, 2023

  1. dont cut, gate it behind LLVM_VERSION_LT

    @klensy
    klensy committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    44678b4 View commit details
    Browse the repository at this point in the history

  2. assert that LLVM_VERSION_* imported before using them

    @klensy
    klensy committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    712b6af View commit details
    Browse the repository at this point in the history

  3. fix warn: llvm-wrapper/RustWrapper.cpp(1254): warning C4244: '=': con… 

    …version from 'uint64_t' to 'unsigned int', possible loss of data

https://github.com/llvm/llvm-project/blame/release/16.x/llvm/include/llvm/IR/DiagnosticInfo.h
3 versions of getLocCookie returns: 1 uint64_t and 2 unsigned, nice consistency.
    @klensy
    klensy committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    8660780 View commit details
    Browse the repository at this point in the history

  4. fix headers

    @klensy
    klensy committed Nov 7, 2023
    Configuration menu
    Copy the full SHA
    380bd4a View commit details
    Browse the repository at this point in the history