RFR: 8253795: Implementation of JEP 391: macOS/AArch64 Port [v12]
Anton Kozlov
akozlov at openjdk.java.net
Tue Mar 9 16:12:37 UTC 2021
On Tue, 9 Feb 2021 09:06:26 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> Anton Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update signal handler part for debugger
>
> src/hotspot/share/runtime/threadWXSetters.hpp line 28:
>
>> 26: #define SHARE_RUNTIME_THREADWXSETTERS_HPP
>> 27:
>> 28: #include "runtime/thread.inline.hpp"
>
> This breaks against our convention to forbid inline.hpp files from being included from being included from .hpp files. You need to rework this by either moving the implementation to a .cpp file, or convert this file into an .inline.hpp
>
> See the Source Files section in:
> https://htmlpreview.github.io/?https://github.com/openjdk/jdk/blob/master/doc/hotspot-style.html
Thanks, I renamed the file to threadWXSetters.inline.hpp
-------------
PR: https://git.openjdk.java.net/jdk/pull/2200
More information about the build-dev
mailing list