In particular we implement the is_same<T,U> templated type. This is
useful for doing compile-time comparison of types in static_asserts.
The plan is to use this in another patch (
https://reviews.llvm.org/D54904 ).
Details
- Reviewers
kcc dvyukov vitalybuka cryptoad eugenis kubamracek george.karpenkov - Commits
- rGc2e2dd43f018: Implement a small subset of the C++ `type_traits` header inside…
rCRT349077: Implement a small subset of the C++ `type_traits` header inside…
rL349077: Implement a small subset of the C++ `type_traits` header inside…
Diff Detail
- Repository
- rL LLVM
Event Timeline
It was removed after https://reviews.llvm.org/D54904?id=175500.
Is this patch still necessary?
@cryptoad
I'd like to still land it if possible. I intend to use this in future patches. I removed it from https://reviews.llvm.org/D54904 to simply have less code to review. My intention is to land this and then add the static_assert(...) that I wanted to write original in a future patch.
@cryptoad
I'd like to still land it if possible. I intend to use this in future patches. I removed it from https://reviews.llvm.org/D54904 to simply have less code to review. My intention is to land this and then add the static_assert(...) that I wanted to write original in a future patch.
Then we can review it with dependent patch?
There is probability that dependent patch will be changed during review or other reasons and we stuck with small but unused piece of code.