[PATCH] D47090: Implement C++17 <memory_resource>.

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 19 11:42:41 PDT 2018


EricWF added inline comments.


================
Comment at: src/memory_resource.cpp:62
+
+namespace {
+
----------------
Quuxplusone wrote:
> EricWF wrote:
> > We certainly don't want a different definition of the global resources in each TU. See below.
> @EricWF: I think all of your comments in this file are the result of misreading "src/memory_resource.cpp" as "include/memory_resource".  Or else I *totally* don't understand the current directory organization and you're going to have to instruct me. :)
> 
Woops. I feel dumb now. 


Repository:
  rCXX libc++

https://reviews.llvm.org/D47090





More information about the cfe-commits mailing list