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

Mon Oct 17 09:24:24 PDT 2022

zahiraam 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
----------------
erichkeane wrote:
> 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.
I did notice that the there is some extra information generated before the attribute at the function define when Ox (x>0) is used.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136084