[PATCH] D74384: Use std::foo_t rather than std::foo in LLVM.

Kadir Çetinkaya via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 10 23:38:48 PST 2020


> I joined the group; what do I do to get tests run for this?

you need to upload a diff to trigger the bots


> > Hope someone can verify the build on Windows.
> Do you think I should go on IRC and ask?

Not much help before committing, but for post-commit there is
http://45.33.8.238/.
Doesn't contain all different configurations but they are fast, so you can
revert swiftly at leat

On Tue, Feb 11, 2020 at 8:25 AM Justin Lebar via Phabricator <
reviews at reviews.llvm.org> wrote:

> jlebar added a comment.
>
> > Such changes can be risky.
>
> Heh, well said.  I feel good about it because I found two or three real
> bugs in clang/llvm as a result of this cleanup.  But still, we've got to
> land it safely...
>
> I joined the group; what do I do to get tests run for this?
>
> > Hope someone can verify the build on Windows.
>
> Do you think I should go on IRC and ask?
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D74384/new/
>
> https://reviews.llvm.org/D74384
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200211/f1dbed98/attachment.html>


More information about the cfe-commits mailing list