Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

r300639 broke self-hosting on ARM Linux #31681

Open Quuxplusone opened 7 years ago

Quuxplusone commented 7 years ago
Bugzilla Link PR32709
Status NEW
Importance P enhancement
Reported by Renato Golin (rengolin@gmail.com)
Reported on 2017-04-19 02:11:14 -0700
Last modified on 2017-04-19 13:33:17 -0700
Version trunk
Hardware PC Linux
CC diana.picus@linaro.org, llvm-bugs@lists.llvm.org, matze@braunis.de, rengolin@gmail.com
Fixed by commit(s)
Attachments bla3.ll (11246 bytes, text/plain)
Blocks
Blocked by
See also
http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15-full-sh/builds/1349

lib/Target/ARM/ARMFrameLowering.cpp:455: virtual void
llvm::ARMFrameLowering::emitPrologue(llvm::MachineFunction &,
llvm::MachineBasicBlock &) const: Assertion `getMaxFPOffset(*MF.getFunction(),
*AFI) <= FPOffset && "Max FP estimation is wrong"' failed.
Quuxplusone commented 7 years ago

Reverted in r300668

Quuxplusone commented 7 years ago

Thanks for reverting this in time and sorry for the breakage. Looks some of my assumptions about how much stuff we have on the stack before FP were wrong.

Quuxplusone commented 7 years ago

Attached bla3.ll (11246 bytes, text/plain): Reproducer