[PATCH] D37887: AMDGPU: Run internalize symbols at -O0

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 14:30:20 PDT 2017


rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:326
+    if (F->isDeclaration() || AMDGPU::isEntryFunctionCC(F->getCallingConv()))
+      return true;
+  }
----------------
arsenm wrote:
> rampitec wrote:
> > Looks like it changes original logic. All functions were internalized except for declarations and kernels.
> It's the same, also no tests fail
If a function is called and not declaration it used to be internalized, with this change it will not.


https://reviews.llvm.org/D37887





More information about the llvm-commits mailing list