[PATCH] D78669: PowerPC: Add emergency stack spill slots for SPE

[Justin Hibbits via Phabricator via llvm-commits]

jhibbits marked an inline comment as done.
jhibbits added a comment.

As mentioned before, I can only seem to reproduce this running natively on powerpcspe; running the test on powerpc64, even without this patch, succeeds.  This is extremely bizarre, but this fix gets the native build to produce the exact same text as running the test on powerpc64.



================
Comment at: llvm/test/CodeGen/PowerPC/spe-spills.ll:424-460
+!llvm.module.flags = !{!0}
+!llvm.ident = !{!1}
+
+!0 = !{i32 1, !"wchar_size", i32 4}
+!1 = !{!"FreeBSD clang version 10.0.1 (git at github.com:llvm/llvm-project.git llvmorg-10.0.1-rc1-0-gf79cd71e145)"}
+!2 = !{!3, !6, i64 0}
+!3 = !{!"", !4, i64 0, !5, i64 40, !9, i64 64, !5, i64 136, !10, i64 160, !11, i64 176}
----------------
One thing to note: these flags are required to reproduce the problem, along with the !tbaa annotations in the code.  Without them, even the otherwise failing llc would succeed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78669