[PATCH] D43627: [DEBUGINFO] Add flag for DWARF2 or less to use sections as references.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 8 06:16:54 PST 2018


dblaikie added subscribers: ABataev, dblaikie.
dblaikie added a comment.

In fact none of this is in the style guide itself, it's just lore and the
usual argument for consistency with surrounding code.

There are many examples of most points on this spectrum - the generally
agreed upon bit is "single line blocks (without connected/related blocks -
like else, etc) shouldn't have braces". Whether you brace multiline, single
statement blocks, or brace all related blocks if one of them is braced, is
much more variable and I generally leave up  to the author unless there's
some really clear consistency argument to be made.


Repository:
  rL LLVM

https://reviews.llvm.org/D43627





More information about the llvm-commits mailing list