[libcxx-commits] [PATCH] D127226: [libc++] Simplify type_traits and use more builtins

[Christopher Di Bella via Phabricator via libcxx-commits]

cjdb added a comment.

In D127226#3611394 <https://reviews.llvm.org/D127226#3611394>, @philnik wrote:

> In D127226#3611365 <https://reviews.llvm.org/D127226#3611365>, @hans wrote:
>
>> Did the reland in https://reviews.llvm.org/rG31ae52859f7a2339f31bb78c5163c23cb872f179 address the issue raised by Alan and Manoj? The commit message only says "Stuff" :-)
>
> Yes I removed these parts for now. The "Stuff" in there wasn't planned, sorry.

In future, would you mind adding something to describe that the problematic parts have been removed please? It'll make it easier to vet the changes because we'll know what to look for.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127226