[flang-commits] [PATCH] D104100: [flang] Add clang-tidy check for braces around if

[Peter Klausler via Phabricator via flang-commits]

klausler added a comment.

In D104100#2816392 <https://reviews.llvm.org/D104100#2816392>, @rovka wrote:

> I disabled the check for Optimizer and Lower. With the newest version of the patch, I'm seeing several violations in lib/Semantics, tools (f18, f18-parse-demo, tco) and 2 rogue ones in the runtime. I'm going to fix the ones from the runtime, but I'm not sure about the others. What's the preferred style for lib/Semantics and tools?

I wouldn't worry about the tools; few if any of those will survive in the llvm tree.  The semantics code uses the stricter style.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104100/new/

https://reviews.llvm.org/D104100