[PATCH] D104100: [flang] Add clang-tidy check for braces around if
Peter Klausler via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 14 08:34:34 PDT 2021
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
More information about the llvm-commits
mailing list