This updates the various classes that support the compliation of Fortran. These classes are shared by the test tools.
LGTM. There are couple of clang-tidy warnings you might wants to go over before pushing.
I thing we get this warning quite often. Is there a difference between header guard in Flang and the one clang-tidy expect?
Do you want to add // namespace mlir?
Yes, it seems to complain about every header file in flang/include/.
It looks like clang-tidy should be fine with this identifier spelling.
Tracking it here: https://github.com/flang-compiler/f18-llvm-project/issues/631
Looks like a sharp edge between clang-format and clang-tidy. clang-format doesn't put it in if there is only 1 declaration. clang-tidy whines about it.