[PATCH] D82766: Compile the RecursiveASTVisitor callbacks test with "/bigobj"

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 29 07:31:29 PDT 2020


gribozavr2 added inline comments.


================
Comment at: clang/unittests/Tooling/CMakeLists.txt:10
 if (MSVC)
+  set_source_files_properties(RecursiveASTVisitorTests/Callbacks.cpp PROPERTIES COMPILE_FLAGS /bigobj)
   set_source_files_properties(RecursiveASTVisitorTest.cpp PROPERTIES COMPILE_FLAGS /bigobj)
----------------
erichkeane wrote:
> Just a nit, but I think we want to alphabetically order these like include files (so after the ExprVisitor below).  
Done, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82766





More information about the cfe-commits mailing list