[PATCH] D136124: [clang][deps] Remove unintentional `move`

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 18 08:17:43 PDT 2022


jansvoboda11 added inline comments.


================
Comment at: clang/test/ClangScanDeps/modules-full-by-mod-name.cpp:18
 //
-// RUN: clang-scan-deps -compilation-database %t_clangcl.cdb -j 4 -format experimental-full \
+// RUN: clang-scan-deps -compilation-database %t_clangcl.cdb -j 1 -format experimental-full \
 // RUN:   -mode preprocess-dependency-directives -module-name=header1 > %t_clangcl.result
----------------
DavidSpickett wrote:
> Would it help to have one be j1 and one j4, any extra coverage by doing that?
Technically that does give us extra coverage. Let's do it if you feel more comfortable that way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136124



More information about the cfe-commits mailing list