> On Apr 22, 2020, at 9:57 AM, Magnus Ihse Bursie > <magnus.ihse.bur...@oracle.com> wrote: > > On 2020-04-22 15:23, Kim Barrett wrote: >> Please review this change to undo the (small) portion of JDK-8238281 >> [1] related to the HotSpot offset_of macro for gcc/clang. That change >> should not have been made. See CR for details. >> >> I did not reinstate the redefinition of offsetof; we shouldn't be >> using that macro, and having the warning enabled will catch >> questionable cases. (I also considered but didn’t poison it.) >> >> CR: >> https://bugs.openjdk.java.net/browse/JDK-8239357 >> >> [1] https://bugs.openjdk.java.net/browse/JDK-8238281 >> >> Webrev: >> https://cr.openjdk.java.net/~kbarrett/8239357/open.00/ > Looks good to me. > > /Magnus
Thanks. >> >> Testing: >> mach5 tier1 >> >> With a temporary change to use offsetof with a non-standard-layout class, >> verified the expected build failures occur with gcc and clang.