[PATCH] D48310: [Power9][NFC] add back-end test case for passing homogeneous fp128 aggregates by value

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 3 11:59:00 PDT 2018


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

Other than a request for comment, LGTM.



================
Comment at: test/CodeGen/PowerPC/f128-aggregates.ll:75
+; Function Attrs: norecurse nounwind readonly
+define fp128 @testStruct_03(%struct.With9fp128params* byval nocapture readonly align 16 %a) {
+; CHECK-LABEL: testStruct_03:
----------------
I don't actually understand this test. Why are all the GPR's being spilled to the stack. Please add a comment explaining it on the commit.


https://reviews.llvm.org/D48310





More information about the llvm-commits mailing list