[RFR]: Force jmethodIDs to be populated under TSAN

Man Cao manc at google.com
Fri Apr 19 18:28:56 UTC 2019


Use "#if INCLUDE_TSAN" instead of "#ifdef INCLUDE_TSAN". Otherwise looks
good.
No need for new webrev.

-Man


On Fri, Apr 19, 2019 at 11:24 AM Jean Christophe Beyler <jcbeyler at google.com>
wrote:

> Looks good :)
> Jc
>
> On Fri, Apr 19, 2019 at 1:22 PM Arthur Eubanks <aeubanks at google.com>
> wrote:
>
> > Force jmethodIDs to be populated under TSAN
> > Otherwise they might be null when we need them.
> >
> > http://cr.openjdk.java.net/~aeubanks/tsanjmethodid/webrev.00/index.html
> >
>
>
> --
>
> Thanks,
> Jc
>


More information about the tsan-dev mailing list