[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