This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Fix for PR36577
ClosedPublic

Authored by samparker on Mar 5 2018, 8:49 AM.

Details

Summary

Don't PerformSHLSimplify if the given node is used by a node that also uses a constant because we may get stuck in an infinite combine loop.

bugzilla: https://bugs.llvm.org/show_bug.cgi?id=36577

Diff Detail

Event Timeline

samparker created this revision.Mar 5 2018, 8:49 AM
thopre added a subscriber: thopre.Mar 5 2018, 9:35 AM
thopre added inline comments.
lib/Target/ARM/ARMISelLowering.cpp
10452

its -> it's while you are at it?

samparker updated this revision to Diff 137135.Mar 6 2018, 1:34 AM

Fixed typo.

This revision is now accepted and ready to land.Mar 6 2018, 5:54 AM
This revision was automatically updated to reflect the committed changes.