[flang-commits] [flang] [flang] Modifications to ieee floating point environment procedures (PR #121949)

[Rainer Orth via flang-commits]

rorth wrote:

I've now retested the latest version on 4 targets, but it's still not right:
- While the patch compiles and all tests `PASS` on both `x86_64-pc-linux-gnu` and `amd64-pc-solaris2.11`
- on both `sparcv9-sun-solaris2.11` and `sparc64-unknown-linux-gnu`, I get the same failures as before:
  ```
  Flang :: Lower/Intrinsics/ieee_femodes.f90
  Flang :: Lower/Intrinsics/ieee_festatus.f90
  ```

This happens because the `REQUIRES: x86-registered-target` doesn't do what you think it does: it checks if `flang` is configured to support the X86 target, which is true in both my targets (configured with `-DLLVM_TARGETS_TO_BUILD=all`).  However, you make no use whatsoever of this information: `bbc` is still invoked without an explicit `-target` option, so runs for the native/default target instead, which is `sparc*-*-*` in this case.

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