[llvm] [AMDPGU] Auto-upgrade ELF mangling in the data layout (PR #163644)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 16 08:15:12 PDT 2025
b-sumner wrote:
> > > > Generally we just update the four .ll files in the device library. Are there other libraries we're concerned about now where a warning wouldn't be desired?
> > >
> > >
> > > It's all the files with a data layout, and if we changed this I suppose there should be auto-upgrade anyway.
> >
> >
> > Aren't there only 4 files? Isn't the warning a good thing in general?
>
> OpenMP currently uses these, HIP has `ockl` in addition I believe.
>
> ```
> "-mlink-builtin-bitcode" "/opt/rocm/amdgcn/bitcode/ocml.bc"
> ...
> ```
For those, I'd say the warning is exposing a problem which should be fixed by updating the 4 .ll files in the device library,
https://github.com/llvm/llvm-project/pull/163644
More information about the llvm-commits
mailing list