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

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 3 13:04:14 PDT 2021


ABataev added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109255



More information about the cfe-commits mailing list