[PATCH] D14340: [DAGCombiner] Improve zextload optimization.

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 11:57:17 PST 2015


arsenm added a subscriber: arsenm.

================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3020-3022
@@ +3019,5 @@
+
+  // Do not change the width of a volatile load.
+  // Do not generate loads of non-round integer types since these can
+  // be expensive (and would be wrong if the type is not byte sized).
+  if (!LoadN->isVolatile() && LoadedVT.bitsGT(ExtVT) && ExtVT.isRound() &&
----------------
Should this check TLI,shouldReduceLoadWidth? It maybe should go along with the other tests before the place this is called


http://reviews.llvm.org/D14340





More information about the llvm-commits mailing list