This is an archive of the discontinued LLVM Phabricator instance.

[InlineCost] Move the code in isGEPOffsetConstant to a lambda. NFC.
ClosedPublic

Authored by eraman on Feb 17 2017, 12:13 PM.

Diff Detail

Event Timeline

eraman created this revision.Feb 17 2017, 12:13 PM

I'm now thinking that this might be better off as part of the main patch and not as a standalone refactoring patch. I'll revert this if you concur.

chandlerc accepted this revision.Feb 24 2017, 12:17 PM

I'm fine if you want this in the main patch, but I actually think this makes sense as a stand-alone refactoring, so LGTM.

This revision is now accepted and ready to land.Feb 24 2017, 12:17 PM

Thanks. Since this looks good to you, I'll go ahead and commit this.

This revision was automatically updated to reflect the committed changes.