[PATCH] D48029: [DAGCombine] Fix alignment for offset loads/stores

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 18 16:22:37 PDT 2018


efriedma added a comment.

The getExtLoad/getTruncStore API seems really confusing, but I guess there's no simple way to fix it.



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:12190
             LD->getMemOperand()->getFlags(), LD->getAAInfo());
         if (NewLoad.getNode() != N)
           return CombineTo(N, NewLoad, SDValue(NewLoad.getNode(), 1), true);
----------------
Can we get rid of this check?


https://reviews.llvm.org/D48029





More information about the llvm-commits mailing list