[flang-commits] [flang] [flang] Build a dedicated fir-test-opt for testing (PR #123473)

[Valentin Clement バレンタイン クレメン via flang-commits]

clementval wrote:

> I do have an issue with `mlir-opt`, now that you've made me aware of it.

So a solution should be found for both. 

`fir-opt` was initially not including conditional code like that. I would have to go back to see why it was added. I would be in favor of getting rid of this instead of building two executable. Anyway `fir-opt` is the one that should be used in the vast majority of lit test. I guess only two tests (the ones that test the conditionally registered path) requires the special `FLANG_INCLUDE_TESTS=ON` version of the tool.
 

https://github.com/llvm/llvm-project/pull/123473