[PATCH] D84069: [NFC][PPC][AIX] Add test coverage for _Complex return values

Xiangling Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 08:19:09 PDT 2020


Xiangling_L added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-complex.ll:25
+
+; CHECK-DAG:   lfs 1,
+; CHECK-DAG:   lfs 2,
----------------
cebowleratibm wrote:
> Note this is lfs and not lfd.  llc appears to be smart enough to use a 4 byte lfs to populate the register when possible.
I guess it's because `llc` is not default to `-O0`, if you add `-O0`, you will get `lfd` back. And that leads me to think that you may want to add optimization level `-O0` since you aimed at testing `optnone` scenario. Also you can test other optimization level if you are confident that the compiler is doing the right thing.


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

https://reviews.llvm.org/D84069





More information about the llvm-commits mailing list