[all-commits] [llvm/llvm-project] 140c29: Clarify a bit the guideline on omitting braces, in...

[Mehdi Amini via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 140c296ef5144136a00b166384c752c37a176879
      https://github.com/llvm/llvm-project/commit/140c296ef5144136a00b166384c752c37a176879
  Author: Mehdi Amini <aminim at google.com>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M llvm/docs/CodingStandards.rst

  Log Message:
  -----------
  Clarify a bit the guideline on omitting braces, including more examples (NFC)

Like most readability rules, it isn't absolute and there is a matter of taste
to it. I think more recent part of the project may be more consistent in the
current application of the guideline. I suspect sources like
mlir/lib/Dialect/StandardOps/IR/Ops.cpp may be examples of this at the moment.

Differential Revision: https://reviews.llvm.org/D82594


  Commit: 221979b6913667dc52b03b181a9349c1b73dbd52
      https://github.com/llvm/llvm-project/commit/221979b6913667dc52b03b181a9349c1b73dbd52
  Author: Mehdi Amini <aminim at google.com>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M llvm/docs/TestingGuide.rst

  Log Message:
  -----------
  Document the testing of Analyses in the LLVM testing guide (NFC)

This came up in a recent review, someone was wondering were was
this all documented and I couldn't find a reference to provide.

Differential Revision: https://reviews.llvm.org/D83816


Compare: https://github.com/llvm/llvm-project/compare/8b85f68ee2dd...221979b69136