[RFR]: Change TSAN_INCLUDE define to LAUNCHER_TSAN_INCLUDE in launcher

Jean Christophe Beyler jcbeyler at google.com
Fri Apr 19 20:27:36 UTC 2019


LGTM

On Fri, Apr 19, 2019 at 3:27 PM Man Cao <manc at google.com> wrote:

> Looks good.
>
> -Man
>
>
> On Fri, Apr 19, 2019 at 1:21 PM Arthur Eubanks <aeubanks at google.com>
> wrote:
>
> >
> http://cr.openjdk.java.net/~aeubanks/tsandeflauncher/webrev.00/index.html
> >
> > Rename INCLUDE_TSAN to LAUNCHER_INCLUDE_TSAN in launcher
> > It's confusing that it's the same name as the hotspot macro but uses a
> > different style (#ifdef vs #if).
> >
>


-- 

Thanks,
Jc


More information about the tsan-dev mailing list