This is an archive of the discontinued LLVM Phabricator instance.

Consistent (non) use of empty lines in include blocks
ClosedPublic

Authored by sbc100 on Feb 20 2018, 1:40 PM.

Details

Summary

The profailing style in lld seem to be to not include such empty lines.
Clang-tidy/clang-format seem to handle this just fine.

Event Timeline

sbc100 created this revision.Feb 20 2018, 1:40 PM
sbc100 updated this revision to Diff 135138.Feb 20 2018, 1:41 PM
  • revert
sbc100 added a reviewer: ruiu.Feb 20 2018, 1:41 PM
ruiu accepted this revision.Feb 20 2018, 1:44 PM

LGTM

As long as it is consistent, I don't care much about this style, and I always use clang-format, so I like this change.

This revision is now accepted and ready to land.Feb 20 2018, 1:44 PM
This revision was automatically updated to reflect the committed changes.