[PATCH] D111525: [fir] Update clang-tidy for the Optimizer directory

[Valentin Clement via Phabricator via llvm-commits]

clementval added a comment.

In D111525#3056893 <https://reviews.llvm.org/D111525#3056893>, @mehdi_amini wrote:

> In D111525#3056452 <https://reviews.llvm.org/D111525#3056452>, @schweitz wrote:
>
>> In D111525#3055658 <https://reviews.llvm.org/D111525#3055658>, @mehdi_amini wrote:
>>
>>> I'm a bit puzzled, many of the values are already defined at the top-level and I don't see an override on the path there. What's the effect of this?
>>>
>>> Please provide correct commit description, just saying `Update .clang-tidy file with the value used in fir-dev.` isn't helpful.
>>
>> Actually, they aren't.  flang/.clang-tidy has a number of deltas to {mlir,llvm}/.clang-tidy,
>
> I looked before commenting, I didn't spot it, can you link more precisely?
>
> For example here is what I see at HEAD:
>
>   $ cat flang/.clang-tidy 
>   Checks: '-llvm-include-order,readability-braces-around-statements,-readability-identifier-naming,-clang-diagnostic-*'
>   InheritParentConfig: true
>
> And at the top level the values are matching what is added in this revision.

What we want to achieve is to have the same clang-tidy behavior than MLIR in these directories since the code here uses a lot of MLIR. We might need to double check whether it is still wise to inherit the config from the parent since the flang/.clang-tidy has a differnt style.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111525