RFR: 8253889: Remove ExecMem constant

Sat Oct 3 05:33:39 UTC 2020

On Thu, 1 Oct 2020 09:36:00 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> > Maybe it did something in the past. Now it just seems to be a complicated way to say "false".
> 
> I think that's a Hotspot style. There are few other places where "symbolic values" for boolean parameters are used. For
> example, `CodeBlobToOopClosure::FixRelocations`. `grep 'static const bool'` identify a few more. I personally find it a
> tad more readable. So, it is not as useless...

Well in that case it is applied very inconsistently since most call sites just use false. Lets see if anyone else has
an opinion on this.

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

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