[flang-commits] [PATCH] D113077: [flang] Add a semantics test for co_min

[Pete Steinfeld via Phabricator via flang-commits]

PeteSteinfeld added a comment.

In D113077#3107084 <https://reviews.llvm.org/D113077#3107084>, @ktras wrote:

> In D113077#3107039 <https://reviews.llvm.org/D113077#3107039>, @PeteSteinfeld wrote:
>
>> In D113077#3107007 <https://reviews.llvm.org/D113077#3107007>, @ktras wrote:
>>
>>> Does `git clang-format` make any changes to fortran code? Or just c/c++ code?
>>
>> It's specific to C/C++.  But I usually run it on both the source code and on any tests written in C/C++ (like some of the runtime tests).
>
> I see, that makes sense. Since Damian's test is in Fortran, the tool won't make any changes then. Does that mean that the style and formatting of code in Fortran tests in `llvm-project` does not have to match any particular style or formatting?

I personally would much prefer that it matches existing tests.  But, as far as I know, there are no automated tools that will flag Fortran formatting inconsistencies.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113077