On 29.12.2024 18:57, Jeremy Drake wrote:
On Sun, 29 Dec 2024, Jacek Caban wrote:
We may need to revert use of register keyword for TEB access or make it
conditional if that's indeed problematic for C++. I couldn't reproduce the
warning with Clang.
I did some playing with godbolt.org. It seems OK with `register` with
`__asm__`: https://godbolt.org/z/xrdn3rG45
but gives an error without:
error: ISO C++17 does not allow 'register' storage class specifier
[-Wregister]
https://godbolt.org/z/cMbhhffhs
(of course, it also gives an error for file-scoped register variable too).
It's good to know that our use of the register keyword is fine and that
the only issue lies with the -Dregister= directive in Doxygen. It seems
Doxygen enables this behavior only for Flex versions older than 2.6.0,
which is now 9 years old. I'm not very familiar with MSYS2, but it
appears the Doxygen package might be missing a Flex build dependency.
Could this explain why this outdated code path is being used?
Thanks,
Jacek
_______________________________________________
Mingw-w64-public mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
