RFR: JDK-8300531: Update MSVC optimization flags to match GCC/Clang [v10]

[Justin King]

I'll work on some benchmark comparisons.

On Sat, Feb 18, 2023, 1:38 PM Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> On Sat, 18 Feb 2023 18:21:44 GMT, Justin King <jcking at openjdk.org> wrote:
>
> >> Update MSVC CFlags to be more consistent with other compilers. Also
> disables RTTI in a simliar manner to GCC/Clang for the JVM.
> >
> > Justin King has updated the pull request incrementally with one
> additional commit since the last revision:
> >
> >   Add comments clarifying -Oy-
> >
> >   Signed-off-by: Justin King <jcking at google.com>
>
> The new title is showing much clearer what you want to achieve, thanks for
> that. But I'd like to re-iterate what Erik said early on:
>
> > Changing the global optimization flags is a pretty big and potentially
> intrusive change. [...] I don't think making the number after the O in the
> flags match what we use for other compilers is motivation enough to warrant
> a change like this.
>
> That is, just making the flags "match" is not motivation enough to accept
> such a change. This needs to be thoroughly tested, and then communicated
> and discussed with the wider JDK community, including but not limited to,
> engineers specifically involved in performance testing, evaluation and
> improvements.
>
> -------------
>
> PR: https://git.openjdk.org/jdk/pull/12073
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/build-dev/attachments/20230219/7d869220/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3999 bytes
Desc: S/MIME Cryptographic Signature
URL: <https://mail.openjdk.org/pipermail/build-dev/attachments/20230219/7d869220/smime.p7s>