[PATCH] D136084: Fix LIT CodeGen/Func-attr.c

[Erich Keane via Phabricator via cfe-commits]

erichkeane added inline comments.


================
Comment at: clang/test/CodeGen/func-attr.c:1
-// RUN: %clang -c -ffast-math -emit-llvm -S -o - %s \
-// RUN: | FileCheck %s
+// RUN: %clang -c -O2 -target x86_64 -ffast-math\
+// RUN: -emit-llvm -S -o - %s  | FileCheck %s
----------------
It isn't really appropriate to add an opt-level to the test, and it doesn't really depend on it.  I don't really understand @saugustine 's request in the other thread: MOST of the clang-codegen tests aren't supposed to have opt-levels added to them, and will fail because of it.

So I'm unshocked that adding '-O2' to a test would cause it to fail.  Clang tests are generally NOT supposed to be run with an opt-setting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136084