[PATCH] D81959: [HIP] Enable -amdgpu-internalize-symbols

[Yaxun Liu via Phabricator via cfe-commits]

yaxunl marked an inline comment as done.
yaxunl added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:64-65
+                        "-shared",
+                        "-mllvm",
+                        "-amdgpu-internalize-symbols",
+                        "-o",
----------------
arsenm wrote:
> 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?
the current approach should work. I will do restructuring later.


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

https://reviews.llvm.org/D81959