[PATCH] D143615: [clang][deps] NFC: Refactor inferred modules test

2023-02-08 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG29b95f05738f: [clang][deps] NFC: Refactor inferred modules test (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D143615: [clang][deps] NFC: Refactor inferred modules test

2023-02-08 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added inline comments. Comment at: clang/test/ClangScanDeps/modules-inferred.m:54 +// CHECK-NEXT: { +// CHECK-NEXT: "context-hash": "{{.*}}", +// CHECK-NEXT: "module-name": "Inferred" benlangmuir wrote: > Why d

[PATCH] D143615: [clang][deps] NFC: Refactor inferred modules test

2023-02-08 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 495986. jansvoboda11 added a comment. Remove unused CDB file Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143615/new/ https://reviews.llvm.org/D143615 Files: clang/test/ClangScanDeps/Inputs/frameworks

[PATCH] D143615: [clang][deps] NFC: Refactor inferred modules test

2023-02-08 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir accepted this revision. benlangmuir added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/ClangScanDeps/modules-inferred.m:54 +// CHECK-NEXT: { +// CHECK-NEXT: "context-hash": "{{.*}}", +// CHECK-NEXT:

[PATCH] D143615: [clang][deps] NFC: Refactor inferred modules test

2023-02-08 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added a reviewer: benlangmuir. Herald added a subscriber: ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch squashes two tests wit