arsenm added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:64-65
+                        "-shared",
+                        "-mllvm",
+                        "-amdgpu-internalize-symbols",
+                        "-o",
----------------
We should probably add an lld driver flag for this rather than relying on the 
backend option? Is there some regular way to specify internalization that could 
add the target pass instead?


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

https://reviews.llvm.org/D81959



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

Reply via email to