[PATCH] D40876: AArch64: Fix emergency spillslot being out of reach for large callframes

Kristof Beyls via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 5 23:49:28 PST 2017


kristof.beyls added a comment.



> I'm still in the process of coming up with a reliable and good testcase. As testcases with hundred of arguments are unwieldy I will probably end up placing a correctness test into the llvm test-suite for this.

FWIW, a while ago, I added the following test to the test-suite to test a whole range of different frame layout aspects. Maybe that test could be extended for the case of a large frame somehow?
Probably, the place to look into extending this test would be to add a new template parameter to function `check_frame_variant`.
http://llvm.org/viewvc/llvm-project/test-suite/trunk/MultiSource/UnitTests/C%2B%2B11/frame_layout/frame_layout.cpp?view=markup


Repository:
  rL LLVM

https://reviews.llvm.org/D40876





More information about the llvm-commits mailing list