On Sun, 22 Sep 2024 13:26:36 GMT, Julian Waters <jwat...@openjdk.org> wrote:

> As I wrote originally: the patch was tested and it works. Moreover, this 
> seems to be the way to do per-file cflag optimization level overrides.

Ah, I remember what I was thinking of now. I was thinking of using 
BUILD_LIBJVM_stackMapTable.cpp_OPTIMIZATION rather than CXXFLAGS. 
Unfortunately, as it turns out, with the exception of Microsoft Visual C++, we 
don't have an optimization level for -O1! As such that approach cannot be used 
in this patch (Unless -O2 also fixes the issue). What a shame, that would have 
been cleaner

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

PR Comment: https://git.openjdk.org/jdk/pull/21119#issuecomment-2366792284

Reply via email to