aaronmondal added subscribers: jhuber6, aaronmondal.
aaronmondal added a comment.

Would it be possible to backport this to Clang 16?

If 
https://github.com/RadeonOpenCompute/ROCm-Device-Libs/commit/8dc779e19cbf2ccfd3307b60f7db57cf4203a5be
 makes it into ROCm 5.5 no distro would be able to build it with "vanilla" 
Clang 16, potentially causing pain for users that try to build ROCm 5.5 with a 
Clang from a package manager (a realistic scenario, considering that one may 
want to invest 5 min to build ROCm but not 40 min to build Clang). ROCm 5.5 
will be the first release to officially support the 7900XT and 7900XTX, so not 
having this potentially causes issues for users with recent AMD hardware. (See 
https://github.com/RadeonOpenCompute/ROCm/issues/1880 for extensive, related 
discussion).

@jhuber6 This wouldn't exactly "solve" 
https://github.com/llvm/llvm-project/issues/60660, but I think this could also 
be a workaround (with potentially better user experience), as allowing users 
build ROCm with regular Clang 16 prevents that deadlock where we can't build 
ROCm anymore. This is entirely based on speculation that ROCm 5.5 won't 
introduce other breakages before its release though, so I'd totally understand 
if this is not a satisfactory solution.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142507/new/

https://reviews.llvm.org/D142507

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to