jyknight added a comment.

The OpenJDK bug was UB in the OpenJDK code: 
https://bugs.openjdk.java.net/browse/JDK-8229258 already fixed in JDK14. (But 
not backported to JDK 11 LTS, which is the version Brooks found an error in 
above.)

They probably need to backport that patch.

My only confusion there is that I had thought GCC had the same optimizations 
around "this" alignment, so it's a bit strange to me that this hadn't been 
causing miscompiles in OpenJDK compiled by GCC, too. (Although, I note the 
OpenJDK build is passing -fno-delete-null-pointer-checks)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99790/new/

https://reviews.llvm.org/D99790

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to