[PATCH] D83437: [PowerPC] Enable default support of quad precision operations

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 10 08:52:07 PDT 2020


nemanjai accepted this revision.
nemanjai added a comment.

LGTM. Thank you.



================
Comment at: llvm/test/CodeGen/PowerPC/float-load-store-pair.ll:71
 ; CHECK-NEXT:    ld 3, a14 at toc@l(3)
-; CHECK-NEXT:    stxvx 34, 1, 5
-; CHECK-NEXT:    li 5, 152
----------------
nemanjai wrote:
> This missing store is a bit of a concern.
Oh NVM. That is a store to pass the f128 on the stack which we don't need to do according to the ABI.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83437





More information about the llvm-commits mailing list