[PATCH] D82502: [PowerPC][Power10] Implement Load VSX Vector and Sign Extend and Zero Extend

Albion Fung via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 08:05:41 PDT 2020


Conanap marked 2 inline comments as done.
Conanap added a comment.

Addressed Anil's comments with regards to the test cases



================
Comment at: llvm/test/CodeGen/PowerPC/p10-vsx-builtins.ll:56
+
+; CHECK: lxvrdx
+; Function Attrs: norecurse nounwind readonly
----------------
anil9 wrote:
> I am not too familiar with the builtins but I never saw a check outside of the two braces in the test cases before,  is it not posible to include it inside the test cases ?
these were extra checks that I should've removed; thanks for catching it


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82502





More information about the llvm-commits mailing list