This is an archive of the discontinued LLVM Phabricator instance.

[NFC][AIX] Remove spaces after the comma for `.csect` directive
ClosedPublic

Authored by Xiangling_L on May 19 2020, 2:28 PM.

Details

Summary

To be consistent with other directives like .comm, '.lcomm', we remove the spaces after the comma for .csect.

Diff Detail

Event Timeline

Xiangling_L created this revision.May 19 2020, 2:28 PM
jasonliu accepted this revision.May 19 2020, 5:35 PM

LGTM.
Although there are some other directives that has comma in them would have spaces, I'm fine if we want to take the position and say "as long as it's AIX specific directive, we will leave no space between commas". (llvm is already inconsistent, we will try to be somewhat consistent within ourselves.)
Thanks for the patch.

This revision is now accepted and ready to land.May 19 2020, 5:35 PM
This revision was automatically updated to reflect the committed changes.