This is an archive of the discontinued LLVM Phabricator instance.

[MC] Remove a stray space after tab for COFF .def directives
ClosedPublic

Authored by mstorsjo on Jan 27 2022, 12:01 PM.

Details

Summary

This extra stray space after tab can be traced back to when printing
of this directive was added originally in
4f01b783a3ca5b2544fe1b1677b5bcf6d91c6d2c. The same commit added
inconsistent printing of space after the ELF .type directive too,
which was fixed later in
77fe07a93aeb8feaf39b7d48ad8fc1e6c9e4c78e.

(This is kind of NFC, but it does alter the output, so it's not
strictly non-functional in that sense.)

Diff Detail

Event Timeline

mstorsjo created this revision.Jan 27 2022, 12:01 PM
mstorsjo requested review of this revision.Jan 27 2022, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 27 2022, 12:01 PM
asl accepted this revision.Jan 27 2022, 12:15 PM

Neat! :)

This revision is now accepted and ready to land.Jan 27 2022, 12:15 PM
This revision was landed with ongoing or failed builds.Jan 28 2022, 1:22 AM
This revision was automatically updated to reflect the committed changes.