[PATCH] D126669: [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp

2022-05-31 Thread Daniel McIntosh via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG35b1cfc76f08: [Driver][Modules] Remove dependence on linking support from… (authored by DanielMcIntosh-IBM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12

[PATCH] D126669: [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp

2022-05-30 Thread Daniel McIntosh via Phabricator via cfe-commits
DanielMcIntosh-IBM added a comment. In D126669#3546172 , @MaskRay wrote: > This change is fine (if I were adding the tests in the first place, I'd add > -c or -S), but I'd like to understand why you have such a toolchain not > supporting linking. > Such

[PATCH] D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules

2022-05-30 Thread Daniel McIntosh via Phabricator via cfe-commits
DanielMcIntosh-IBM added inline comments. Comment at: clang/test/Driver/modules.cpp:79-86 +// RUN: %clang++ -fcxx-modules -std=c++17 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX17-MODULES +// CHECK-CXX17-MODULES: "-fcxx-modules" +// RUN: %clang++ -fcxx-modules -std=c++14

[PATCH] D126669: Remove dependence on linking support from clang/test/Driver/modules.cpp

2022-05-30 Thread Daniel McIntosh via Phabricator via cfe-commits
DanielMcIntosh-IBM created this revision. DanielMcIntosh-IBM added reviewers: MaskRay, ChuanqiXu, iains, jansvoboda11, Bigcheese, rsmith. Herald added a subscriber: StephenFan. Herald added a project: All. DanielMcIntosh-IBM requested review of this revision. Herald added a project: clang. Herald

[PATCH] D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules

2022-05-30 Thread Daniel McIntosh via Phabricator via cfe-commits
DanielMcIntosh-IBM added inline comments. Comment at: clang/test/Driver/modules.cpp:79-86 +// RUN: %clang++ -fcxx-modules -std=c++17 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX17-MODULES +// CHECK-CXX17-MODULES: "-fcxx-modules" +// RUN: %clang++ -fcxx-modules -std=c++14

[PATCH] D97796: [test] Fix apparent typo in clang/test/Driver/std.c

2021-03-02 Thread Daniel McIntosh via Phabricator via cfe-commits
DanielMcIntosh-IBM created this revision. DanielMcIntosh-IBM added a reviewer: thakis. DanielMcIntosh-IBM requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Currently the test on line 3 is identical to the test on line 1. Looking at the rest o