[PATCH] D45995: [DAGCombiner] Set the right SDLoc on a newly-created zextload

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 23 15:49:59 PDT 2018


aprantl added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:8032
 
       ExtendSetCCUses(SetCCs, N0, ExtLoad, SDLoc(N), ISD::ZERO_EXTEND);
       // If the load value is used only by N, replace it via CombineTo N.
----------------
Out of curiosity, why doesn't this also get the `SDLoc(N0)`?


https://reviews.llvm.org/D45995





More information about the llvm-commits mailing list