On Tue, 13 Dec 2022 00:37:42 GMT, David Holmes wrote:
>> Primarily, but its not a requirement. We should also be able to invoke
>> `java` as is. The environment variables should be used to force specific
>> behavior for a single invocation. Otherwise, if one forgets to pass the
>> environment
On Mon, 12 Dec 2022 08:04:57 GMT, Justin King wrote:
>> src/java.base/share/native/launcher/main.c line 41:
>>
>>> 39: // extremely early during library loading, before main is called. We
>>> need to override the default
>>> 40: // options because by default UBSan only prints a warning for each
On Mon, 12 Dec 2022 09:58:54 GMT, Justin King wrote:
>> src/java.base/share/native/launcher/main.c line 49:
>>
>>> 47: #endif // UNDEFINED_BEHAVIOR_SANITIZER
>>> 48:
>>> 49: /*
>>
>> As I said we have more launcher than 'java', if you put this method here you
>> must put it in all launchers.
On Mon, 12 Dec 2022 08:14:51 GMT, Robbin Ehn wrote:
>> Justin King has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add comment explaining __ubsan_default_options and UBSAN_OPTIONS
>>
>> Signed-off-by: Justin King
>
> src/java.base/
On Mon, 12 Dec 2022 07:02:04 GMT, Justin King wrote:
>> Allow building OpenJDK with UBSan. Currently the build fails when optimizing
>> the image due to lots of undefined behavior (it invokes the built JVM).
>> Follow up PRs will either replace the undefined behavior with well defined
>> behav
On Mon, 12 Dec 2022 07:26:21 GMT, David Holmes wrote:
>> Justin King has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add comment explaining __ubsan_default_options and UBSAN_OPTIONS
>>
>> Signed-off-by: Justin King
>
> src/java.bas
On Mon, 12 Dec 2022 07:02:04 GMT, Justin King wrote:
>> Allow building OpenJDK with UBSan. Currently the build fails when optimizing
>> the image due to lots of undefined behavior (it invokes the built JVM).
>> Follow up PRs will either replace the undefined behavior with well defined
>> behav
On Mon, 12 Dec 2022 06:47:44 GMT, David Holmes wrote:
>> Yeah, it is unfortunate. However there is no other way to actually set the
>> defaults nicely. The other alternative is to use environment variables, but
>> they are easy to forget when invoking the launcher manually.
>
> Does the env-var
> Allow building OpenJDK with UBSan. Currently the build fails when optimizing
> the image due to lots of undefined behavior (it invokes the built JVM).
> Follow up PRs will either replace the undefined behavior with well defined
> behavior or suppress errors which are intentional. The goal is t
