[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 15:00:20 PST 2025


clementval wrote:

> > > No, we aren't. We are just enabling it, then running tests, then running the install target.
> > 
> > 
> > So the executable for fir-opt is always the same. Like if there is no condition. I'm even more confused now to understand where the problem is.
> 
> Sorry, let me rephrase. There are two possible build pipelines:
> 
> 1. Pipeline with testing: we build with tests enabled, run tests, install.
> 2. Pipeline without testing: we build with tests disabled, install.
> 
> Both pipelines must produce the same result.

Ok now that makes more sense to me. So in order to make this move forward I would suggest to use the newly created executable only for the alias analysis tests and `flang/test/Fir/OpenACC/openacc-mappable.fir` so that we modify less file. Ultimately would should find a way to have a single executable but that might require more time. 

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


More information about the flang-commits mailing list