[PATCH] D78554: [TSAN] Add optional support for distinguishing volatiles

[Dmitry Vyukov via Phabricator via llvm-commits]

dvyukov added a comment.

In D78554#1994518 <https://reviews.llvm.org/D78554#1994518>, @melver wrote:

> In D78554#1994478 <https://reviews.llvm.org/D78554#1994478>, @dvyukov wrote:
>
> > I am not opposed to adding this in general. Andrey did the same for KTSAN in gcc long time ago.
> >  However, this feature will be present only in the newest clang and not in gcc. So what is the use story for kernel? Do we make KCSAN require the newest clang? I think majority of kernel developers use gcc.
>
>
> I am planning to make the same change to gcc. And yes, it's a problem that it's only the newest versions that will support this. But, for now it's not urgent and no kernel dev is explicitly requesting this. But my feeling is that eventually we will need it, and we will be glad to have this option when that day comes.


Ack. At the very least we will be able to use this on syzbot.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78554