[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