This is an archive of the discontinued LLVM Phabricator instance.

[DAGCombiner] Fix SDLoc in a (sext (sextload x)) combine (3/N)
ClosedPublic

Authored by vsk on Apr 27 2018, 6:10 PM.

Details

Summary

Prior to this patch, for the given test case, we would apply the
location associated with the sdiv instruction to instructions which
perform the load.

Part of: llvm.org/PR37262

Diff Detail

Event Timeline

vsk created this revision.Apr 27 2018, 6:10 PM
aprantl accepted this revision.Apr 28 2018, 8:54 AM

That looks obviously correct. Thanks!

This revision is now accepted and ready to land.Apr 28 2018, 8:54 AM
vsk retitled this revision from [DAGCombiner] Fix SDLoc in a (sext (sextload x)) combine (6/N) to [DAGCombiner] Fix SDLoc in a (sext (sextload x)) combine (3/N).May 1 2018, 12:50 PM
This revision was automatically updated to reflect the committed changes.