tra added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:116
+ if (getOrCheckAMDGPUCodeObjectVersion(C.getDriver(), Args) >= 4)
+ OffloadKind = OffloadKind + "v4";
for (const auto &II : Inputs) {
----------------
We do not do it for v2/v3. Could you elaborate on what makes v4 special that it
needs its own offload kind?
Will you need to target different object versions simultaneously?
If yes, how? AFAICT, the version specified is currently global and applies to
all sub-compilations.
If not, then do we really need to encode the version in the offload target name?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99235/new/
https://reviews.llvm.org/D99235
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
