[libcxx-commits] [PATCH] D118056: [libc++][nfc] Include test_macros.h in more tests.

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jan 24 12:38:04 PST 2022


ldionne added inline comments.


================
Comment at: libcxx/test/std/concepts/concepts.lang/concept.assignable/assignable_from.compile.pass.cpp:26
 #include "MoveOnly.h"
+#include "test_macros.h"
 
----------------
I don't understand why this is needed in several of the places where you're including the file. For example, here, I don't see anything that uses a macro in `test_macros.h`. Am I missing it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118056



More information about the libcxx-commits mailing list