Skip to content

Comments

Support LLVM 19 and ROCm 6.4.x#240

Merged
ggeorgakoudis merged 4 commits intomainfrom
update-llvm-19
Aug 5, 2025
Merged

Support LLVM 19 and ROCm 6.4.x#240
ggeorgakoudis merged 4 commits intomainfrom
update-llvm-19

Conversation

@ggeorgakoudis
Copy link
Collaborator

  • Guard parallel split codegen for LLVM 18 because it requires an interface exported only in that version
  • Update lambda handling as LLVM 19 can generate offsets on a byte pointer to access the lambda class fields
  • Other small code fixes for compatibility with LLVM 19
  • Add testing for LLVM 19 and ROCm 6.4.1

- Guard parallel split codegen for LLVM 18 because it requires an interface
exported only in that version
- Update lambda handling as LLVM 19 can generate offsets on a byte pointer
to access the lambda class fields
- Other small code fixes for compatibility with LLVM 19
- Add testing for LLVM 19 and ROCm 6.4.1
@ggeorgakoudis ggeorgakoudis changed the title Support LLVM 19 and ROCm 6.4.1 Support LLVM 19 and ROCm 6.4.x Aug 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 5, 2025

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v18.1.3) reports: 2 concern(s)
  • include/proteus/CoreLLVMHIP.hpp:28:10: error: [clang-diagnostic-error]

    'lld/Common/Driver.h' file not found

       28 | #include <lld/Common/Driver.h>
          |          ^~~~~~~~~~~~~~~~~~~~~
  • include/proteus/Error.h:4:1: warning: [llvm-include-order]

    #includes are not sorted properly

        4 | #include <cstdint>
          | ^        ~~~~~~~~~
          |          <llvm/ADT/Twine.h>
        5 | #include <llvm/ADT/Twine.h>
          |          ~~~~~~~~~~~~~~~~~~
          |          <cstdint>

Have any feedback or feature suggestions? Share it here.

@ggeorgakoudis ggeorgakoudis merged commit 4a2df1e into main Aug 5, 2025
24 checks passed
@ggeorgakoudis ggeorgakoudis deleted the update-llvm-19 branch August 5, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants