This is an archive of the discontinued LLVM Phabricator instance.

[AArchExpandPseudo] Preserve instruction debug number in expansions
ClosedPublic

Authored by fdeazeve on Mar 13 2023, 7:54 AM.

Details

Summary

This is an initial attempt at preserving debug information in the pseudo
instruction expansion of the AArch backend. In particular, we preserve
the instruction number required by the InstrRef implementation of live
debug values.

There are many other expansions that need to be considered, but the ones
addressed in this commit should be extremely common, as they handle most
arithmetic and logical instructions.

Diff Detail

Event Timeline

fdeazeve created this revision.Mar 13 2023, 7:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2023, 7:54 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
fdeazeve requested review of this revision.Mar 13 2023, 7:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2023, 7:54 AM
jmorse accepted this revision.Mar 13 2023, 8:19 AM

LGTM, assuming that the instructions affected always have the same signature / format. (I know very little aarch64).

This revision is now accepted and ready to land.Mar 13 2023, 8:19 AM
aprantl accepted this revision.Mar 13 2023, 11:56 AM
This revision was landed with ongoing or failed builds.Mar 13 2023, 12:29 PM
This revision was automatically updated to reflect the committed changes.