RFR: 8334618: ubsan: support setting additional ubsan check options [v3]
Matthias Baesken
mbaesken at openjdk.org
Sat Jun 22 09:02:10 UTC 2024
On Fri, 21 Jun 2024 23:07:57 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
> [Just a drive-by comment, not a review and not planning to review.] As a user, syntactically I think I'd prefer something like the `with-ubsan[=parameters]` approach suggested in the the probably duplicate issue https://bugs.openjdk.org/browse/JDK-8306710. But that's not a strongly held opinion. And I have no idea what would be better for the implementation.
Your idea is more flexible (you can not just add something but overwrite the existing settings).
However you have to set more ubsan parameters on the other hand, so both approaches have pros and cons.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/19802#issuecomment-2183951780
More information about the build-dev
mailing list