[PATCH] D125506: [PowerPC] Implement XL compat __fnabs and __fnabss builtins.

[Lei Huang via Phabricator via cfe-commits]

lei added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/builtins-ppc-xlcompat-fnabs.ll:46
+; RUN:   -mattr=-vsx < %s | FileCheck %s --check-prefix=CHECK-NOVSX
+
+declare double @llvm.ppc.fnabs(double)
----------------
nit: same as before no need to test all combination for both pwr7 and pwr8.  Just a mix of the 2 like in the clang test. What happens if we do pwr6 with `+vsx`?  Is this a possible combination?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125506