[PATCH] D54358: [AMDGPU] Disable DAG combine at -O0

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 14:57:18 PST 2018


rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:8581-8582
                                             DAGCombinerInfo &DCI) const {
+  if (getTargetMachine().getOptLevel() == CodeGenOpt::None)
+    return SDValue();
+
----------------
arsenm wrote:
> I don't see why we would bother disabling specifically the target combines at -O0, but not all of them. I thought combines were already partially skipped at -O0?
I do not see it. Moreover, I see that combiner works regardless of optimization level.


https://reviews.llvm.org/D54358





More information about the llvm-commits mailing list