JIT: Fix compiler warning from visibility attribute in typedef#139981
Conversation
There was a problem hiding this comment.
LGTM. Basically the same as my PR #140286 I've just closed since @Fidget-Spinner made me aware of yours :)
Just a suggestion:
|
Oh, I think I've found the answer myself: adding |
|
Maybe also restore the comment that got lost in #137961? Because it is utterly important that we use |
|
Thanks @savannahostrowski for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14. |
|
Sorry, @savannahostrowski, I could not cleanly backport this to |
|

This was introduced in #136528. The
DECLARE_TARGETmacro includesvisibility("hidden"), which is valid for forward declarations but gets ignored when used in a typedef, so the compiler is yelling.