-
Notifications
You must be signed in to change notification settings - Fork 10.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[flang][driver] support symbol visibility flag #92459
Comments
@llvm/issue-subscribers-flang-driver Author: None (h-vetinari)
While working on upgrading the SciPyb-builds in conda-forge to use flang 18, I'm running into the fact that flang got a lot stricter about unknown flags (which previously were just ignored, presumably).
Some of this probably needs to be fixed on the meson side (which generates the actual compiler invocations from the build files), but I noticed one case where there's both arguably missing functionality, and the warnings don't make sense, namely SciPy specifies hidden visibility for some targets, and when this gets hit by flang (on windows), the result at first is
Using the suggested
Finally, using
At the very least the warnings are misleading. But setting symbol visibility should probably be supported in general? |
While working on upgrading the SciPyb-builds in conda-forge to use flang 18, I'm running into the fact that flang got a lot stricter about unknown flags (which previously were just ignored, presumably).
Some of this probably needs to be fixed on the meson side (which generates the actual compiler invocations from the build files), but I noticed one case where there's both arguably missing functionality, and the warnings don't make sense, namely
-fvisibility=
, which clang documents as:SciPy specifies hidden visibility for some targets, and when this gets hit by flang (on windows), the result at first is
Using the suggested
-Xclang -fvisibility=hidden
, I then getFinally, using
-Xflang -fvisibility=hidden
, the result isAt the very least the warnings are misleading. But setting symbol visibility should probably be supported in general?
The text was updated successfully, but these errors were encountered: