[libcxx-commits] [PATCH] D153216: [libc++][Modules] Add missing includes to public headers

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/include/memory_resource:53
 #include <__config>
+#include <__fwd/memory_resource.h>
 #include <__memory_resource/memory_resource.h>
----------------
ldionne wrote:
> Yes!
IMO `__memory_resource/memory_resource.h` should include the forward declaration, not the top-level header.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153216