RFR: 8281467: Allow larger OptoLoopAlignment and CodeEntryAlignment

Harold Seigel hseigel at openjdk.java.net
Wed Feb 9 18:58:05 UTC 2022


On Wed, 9 Feb 2022 06:51:47 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> src/hotspot/share/runtime/globals.hpp line 1539:
>> 
>>> 1537:           range(1, 128)                                                     \
>>> 1538:           constraint(OptoLoopAlignmentConstraintFunc, AfterErgo)            \
>>> 1539:                                                                             \
>> 
>> Should OptoLoopAlignment be an int, instead of an intx, since its range is small?
>
> Dunno, maybe? I see the lot of other "small" options are `intx`, and the change like that would proliferate to all architectures that set `OptoLoopAlignment` as their `product_pd`. It also raises the question if `CodeEntryAlignment` should also be `int`? I'd rather keep this patch small, to be honest.

Your comment makes sense.  Thanks.

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

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


More information about the hotspot-dev mailing list