[PATCH] D100669: [ADT] Add llvm::remove_cvref and llvm::remove_cvref_t

[Scott Linder via Phabricator via llvm-commits]

scott.linder added inline comments.


================
Comment at: llvm/unittests/ADT/STLForwardCompatTest.cpp:48-52
+using STLForwardCompatRemoveCVRefTestTypes = ::testing::Types<
+    std::pair<int, int>, std::pair<int &, int>, std::pair<const int, int>,
+    std::pair<volatile int, int>, std::pair<const volatile int &, int>,
+    std::pair<int *, int *>, std::pair<int *const, int *>,
+    std::pair<const int *, const int *>, std::pair<int *&, int *>>;
----------------
The formatting here gets a bit muddy, I considered adding a typedef of `std::pair` with a longer name, so that the formatting would refer one entry per line. Does that seem like it would be an improvement?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100669