[PATCH] D149206: [clang][driver] Enable MisExpect diagnostics flag outside of CC1
Paul Kirth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 26 08:29:37 PDT 2023
paulkirth added inline comments.
================
Comment at: clang/test/Profile/misexpect-branch.c:10
// RUN: %clang_cc1 %s -O2 -o - -emit-llvm -fprofile-instrument-use-path=%t.profdata -verify=foo -fdiagnostics-misexpect-tolerance=10 -Wmisexpect -debug-info-kind=line-tables-only
+// RUN: %clang -c -S %s -O2 -o - -emit-llvm -fprofile-instr-use=%t.profdata -Xclang -verify=foo -fdiagnostics-misexpect-tolerance=10 -Wmisexpect -gline-tables-only
// RUN: %clang_cc1 %s -O2 -o - -disable-llvm-passes -emit-llvm -fprofile-instrument-use-path=%t.profdata -verify=foo
----------------
hans wrote:
> The more common way to test this would be a test under clang/test/Driver/ that just checks that forwarding the flag works.
Good point, I just defaulted to adding the RUN line here, but that's definitely the right way to test this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149206/new/
https://reviews.llvm.org/D149206
More information about the cfe-commits
mailing list