This is an archive of the discontinued LLVM Phabricator instance.

[VE] Restructure eliminateFrameIndex
ClosedPublic

Authored by kaz7 on Jul 1 2022, 10:16 PM.

Details

Summary

Restructure the current implementation of eliminateFrameIndex function
in order to support more instructions.

Diff Detail

Event Timeline

kaz7 created this revision.Jul 1 2022, 10:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 1 2022, 10:16 PM
kaz7 requested review of this revision.Jul 1 2022, 10:16 PM
kaz7 updated this revision to Diff 441877.Jul 2 2022, 3:54 AM

Add regression tests and support the case like the first 8 bytes fits in
the immediate field but the second 8 bytes doesn't fit in the immediate
field.

efocht accepted this revision.Jul 5 2022, 2:36 AM

Looks good to me.

This revision is now accepted and ready to land.Jul 5 2022, 2:36 AM
This revision was landed with ongoing or failed builds.Jul 5 2022, 3:28 AM
This revision was automatically updated to reflect the committed changes.
kaz7 reopened this revision.Jul 5 2022, 3:44 AM
This revision is now accepted and ready to land.Jul 5 2022, 3:44 AM
kaz7 updated this revision to Diff 442250.Jul 5 2022, 3:44 AM

Add missing DEBUG_TYPE.

This revision was landed with ongoing or failed builds.Jul 5 2022, 4:00 AM
This revision was automatically updated to reflect the committed changes.