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

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 10 23:24:57 PST 2020


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





More information about the llvm-commits mailing list