zahiraam added a comment. Looks good to me after the additional RUN line mentioned in my comment for Driver/fp-model.c is added. Thanks.
================ Comment at: clang/test/Driver/fp-model.c:77 +// WARN12-NOT: warning: overriding '-ffp-model=strict' option with '-ffp-model=strict' [-Woverriding-t-option] + // RUN: %clang -### -c %s 2>&1 \ ---------------- Please add this: +// RUN: %clang -### -Ofast -ffp-model=strict -c %s 2>&1 | FileCheck \ +// RUN: --check-prefix=WARN12 %s +// WARN12-NOT: warning: overriding '-ffp-model=strict' option with '-ffp-model=strict' [-Woverriding-t-option] Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137618/new/ https://reviews.llvm.org/D137618 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits