[flang-commits] [flang] [flang] Adjust _FORTRAN_RUNTIME_IEEE_FENV_T_EXTENT for Solaris (PR #74590)

[via flang-commits]

vdonaldson wrote:

> Just removing the assertion fixes the Solaris build indeed: the testsuite results are identical to those with my patch.
> 
> How should we do this properly: just add a comment in `exceptions.cpp` or also add one in `magic-numbers.h` indicating that `_FORTRAN_RUNTIME_IEEE_FENV_T_EXTENT` is too small on Solaris?

There is already a comment at the end of `exceptions.cpp` for `femode_t` objects, which is very similar to th. My this problem. My preference is to add/extend the comment there.

My original code for this was an assert in the compiler rather than the runtime. That nicely solves all existing issues. The problem is that we would eventually like to support cross compilation, where a front end assert doesn't work. One way to resolve this would be to allocate these objects dynamically at runtime, although that would penalize typical compilations. Eventually we might want to generate different code for local compilation vs. cross compilation.

Bottom line - I'm still looking at this. For now I can live with something that allows builds to succeed.

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