This is an archive of the discontinued LLVM Phabricator instance.

[X86]: Adding test cases for LEA factorization.
ClosedPublic

Authored by jbhateja on Jul 26 2017, 5:33 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

jbhateja created this revision.Jul 26 2017, 5:33 AM
jbhateja updated this revision to Diff 108253.Jul 26 2017, 5:36 AM

Remving an NFC spell correction.

jbhateja accepted this revision.Jul 26 2017, 5:37 AM
This revision is now accepted and ready to land.Jul 26 2017, 5:37 AM
jbhateja resigned from this revision.Jul 26 2017, 5:44 AM
jbhateja added a reviewer: RKSimon.

LGTM.

This revision now requires review to proceed.Jul 26 2017, 5:44 AM
jbhateja accepted this revision.Jul 26 2017, 5:44 AM
This revision is now accepted and ready to land.Jul 26 2017, 5:44 AM

Simon, can you kindly land this for me into trunk. Thanks.

This revision was automatically updated to reflect the committed changes.
RKSimon edited edge metadata.Jul 27 2017, 3:38 AM

Simon, can you kindly land this for me into trunk. Thanks.

Done - I'm sorry but I realised as I sent it that I didn't include a 'committed on the behalf' tag in the commit message.