[PATCH] D109255: [NFC][OpenMP] Add fsyntax-only to driver tests

Fri Sep 3 13:07:58 PDT 2021

jsji added a comment.

In D109255#2983052 <https://reviews.llvm.org/D109255#2983052>, @ABataev wrote:

> In D109255#2983017 <https://reviews.llvm.org/D109255#2983017>, @jdenny wrote:
>
>> This test is meant to check that the driver processes these options correctly.  -cc1 isn't the driver and often ignores either the positive or negative version of an option.
>
> Better not to do such checks, just need to check that driver correctly translates driver options to the frontend options (using `-###`), better to use `-verify` only with the frontend tests. Suggest to rework this test.

Make sense to me. How about we use `_cc1` as Alexey suggested here. Maybe Joel can add another test to test the driver correctly translates driver options to the frontend options (using `-###`) in `test/Driver` dir?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109255