RFR: 8346193: Test runtime/ErrorHandling/TestDwarf.java fails build with clang17 [v2]

SendaoYan syan at openjdk.org
Wed Dec 18 12:59:39 UTC 2024


On Wed, 18 Dec 2024 09:47:23 GMT, Hamlin Li <mli at openjdk.org> wrote:

> the current fix depends on the internal implementation and the effect of a compilation flag of clang, this is not stable

I don't think the current fix is not stable. According the cppreference documentation:

1.  [volatile accesses cannot be optimized out](https://en.cppreference.com/w/cpp/language/cv) 
2. [Accesses (reads and writes) to volatile objects occur strictly according to the semantics of the expressions in which they occur.](https://en.cppreference.com/w/cpp/language/as_if)

> seems the test does not check the crash details

Yes, use `ShouldNotReachHere();` instead of `*t = 'c';`, the test still run passed which tested jdk build from gcc. From this perspective, use `ShouldNotReachHere();` also acceptable.

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

PR Comment: https://git.openjdk.org/jdk/pull/22757#issuecomment-2551257350


More information about the hotspot-dev mailing list