This is an archive of the discontinued LLVM Phabricator instance.

[Debug-Info] Add names for the debug line prologue.
ClosedPublic

Authored by Esme on Mar 10 2021, 7:18 PM.

Details

Summary

This is a minor patch to add names for the debug line prologue, as a follow-up of D95998.

Diff Detail

Event Timeline

Esme created this revision.Mar 10 2021, 7:18 PM
Esme requested review of this revision.Mar 10 2021, 7:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2021, 7:18 PM
dblaikie accepted this revision.Mar 10 2021, 7:30 PM

Looks good

This revision is now accepted and ready to land.Mar 10 2021, 7:30 PM
shchenz accepted this revision.Mar 10 2021, 7:32 PM

Thanks for doing this. LGTM too.

ikudrin accepted this revision.Mar 10 2021, 9:24 PM

LGTM, thanks!

llvm/lib/MC/MCDwarf.cpp
478–481

These two symbols make a pair, wouldn't it be better to put them under a common comment?

This revision was landed with ongoing or failed builds.Mar 11 2021, 8:46 PM
This revision was automatically updated to reflect the committed changes.