[PATCH] D137618: [Clang] Fix behavior of -ffp-model option when overriden

[Zahira Ammarguellat via Phabricator via cfe-commits]

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