benlangmuir added inline comments.
Comment at: clang/test/Driver/compilation_database_fsyntax_only.c:1
+// RUN: mkdir -p %t.workdir && cd %t.workdir
+// RUN: %clang -fsyntax-only %s -MJ - 2>&1 | FileCheck %s
MaskRay wrote:
> The test can be added into `compilatio
This revision was automatically updated to reflect the committed changes.
Closed by commit rGaca23d8ac30d: [clang] Fix assertion failure using -MJ with
-fsyntax-only (authored by benlangmuir).
Changed prior to commit:
https://reviews.llvm.org/D159016?vs=554046&id=554765#toc
Repository:
rG LL
MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.
Looks great!
Comment at: clang/test/Driver/compilation_database_fsyntax_only.c:1
+// RUN: mkdir -p %t.workdir && cd %t.workdir
+// RUN: %clang -fsyntax-only %s -MJ - 2>&1 |
benlangmuir created this revision.
benlangmuir added a reviewer: MaskRay.
Herald added a project: All.
benlangmuir requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
If there is no output filename we should not assert when writing output for -