Page MenuHomePhabricator

[X86] Merge disp8 and cdisp8 handling into a single helper function to reduce some code.
ClosedPublic

Authored by craig.topper on Mon, Jul 27, 11:52 PM.

Details

Summary

We currently handle EVEX and non-EVEX separately in two places. By sinking the EVEX
check into the existing helper for CDisp8 we can simplify these two places.

Diff Detail

Event Timeline

craig.topper created this revision.Mon, Jul 27, 11:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptMon, Jul 27, 11:52 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Jul 28, 12:36 AM
Harbormaster failed remote builds in B65964: Diff 281124!
craig.topper requested review of this revision.Tue, Jul 28, 6:04 AM
RKSimon accepted this revision.Tue, Jul 28, 8:28 AM

LGTM

This revision is now accepted and ready to land.Tue, Jul 28, 8:28 AM
This revision was landed with ongoing or failed builds.Tue, Jul 28, 10:46 AM
This revision was automatically updated to reflect the committed changes.