[PATCH] D58421: Add partial implementation of std::to_address() as llvm::to_address()

Glen Fernandes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 20 16:57:55 PST 2019


glenjofe added inline comments.


================
Comment at: llvm/trunk/include/llvm/ADT/STLExtras.h:1588
+template <class Ptr> auto to_address(const Ptr &P) -> decltype(P.operator->()) {
+  return P.operator->();
+}
----------------
The fancy pointer overload should return `to_address(p.operator->())` but this returns `p.operator->()`.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58421





More information about the llvm-commits mailing list