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

[Mehdi AMINI via Phabricator via flang-commits]

mehdi_amini added a comment.

In D104100#2817259 <https://reviews.llvm.org/D104100#2817259>, @mehdi_amini wrote:

> In D104100#2813793 <https://reviews.llvm.org/D104100#2813793>, @klausler wrote:
>
>> This should probably not be applied to all of f18; the lowering code, which interacts heavily with mlir and llvm, respects their styles.
>
> I thought not having `readability-braces-around-statements` was the llvm/mlir default?

Ah I misunderstood what the current patch is doing...


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

https://reviews.llvm.org/D104100