[PATCH] D108555: [tsan] Do not include <stdatomic.h> from sanitize-thread-disable.c

Marco Elver via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 23 07:18:53 PDT 2021


melver added a comment.

In D108555#2960037 <https://reviews.llvm.org/D108555#2960037>, @glider wrote:

> In D108555#2960034 <https://reviews.llvm.org/D108555#2960034>, @melver wrote:
>
>> LGTM, thanks!
>>
>> Patch title ("...an X86-only test..") also needs adjustment.
>
> It's strange that Phab doesn't automatically update the title when I update the commit message.

Hmm, yeah I remember this was broken before. I guess as long as the committed version is right it's fine. I think it's also possible to edit on web interface (not so nice..) and use arc amend or something.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108555



More information about the cfe-commits mailing list