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

Valentin Clement バレンタイン クレメン via flang-commits flang-commits at lists.llvm.org
Mon Jan 20 09:17:15 PST 2025


clementval wrote:

`fir-opt` has the exact same purpose than `mlir-opt` and just include the dialects that are specific to flang. If you don't have an issue with `mlir-opt` then there should not be one with `fir-opt`. 

> This is a QA policy violation.

Policy that comes from where?

> I still do not understand this. If the file is not supposed to be installed, why is it installed? And if it is only needed for testing, why is there a different variant for non-test scenario in the first place?

Well it's an llvm executable and they are all installed. Like all the `mlir-xxx` tools. It might be useful for some users to be able to execute flang passes with such a tool but I see it more as a dev tools than a lambda user tool. 

I'm still confused of what you are trying to solve if you don't have any error. Can you describe your process and where exactly it poses a problem?




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


More information about the flang-commits mailing list