[PATCH] D29214: [AMDGPU] Internalize non-kernel symbols

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:88-90
+static cl::opt<bool> WholeProgram(
+  "amdgpu-whole-program",
+  cl::desc("Enable whole program mode"),
----------------
This isn't a good name (and shouldn't it default to true?). How about amdgpu-internalize-functions/


Repository:
  rL LLVM

https://reviews.llvm.org/D29214