On Sat, 11 Dec 2021 14:05:12 GMT, Andrew Haley <a...@openjdk.org> wrote:

>> Alan Hayward has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Remove BSD/Apple specific code
>
> src/hotspot/cpu/aarch64/globals_aarch64.hpp line 122:
> 
>> 120:           "It cannot be used with OnSpinWaitInst=none.")                
>> \
>> 121:           range(1, 99)                                                  
>> \
>> 122:   product(bool, UseROPProtection, false,                                
>> \
> 
> Question: this is called "UseROPProtection", the configure option is called 
> "enable-branch-protection", and GCC option is called "-mbranch-protection". 
> This is confusing. I would have thought we would want the same name, and use 
> it for all branch protection. So why is this not "UseBranchProtection"?

`-mbranch-protection` switches on both PAC-RET and BTI. This PR only covers a 
use of PAC that looks very ROP-focused to me.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6334

Reply via email to