[PATCH] D18619: [AArch64] Combine callee-save and local stack SP adjustment instructions.
Tim Northover via llvm-commits
llvm-commits at lists.llvm.org
Tue May 3 10:35:12 PDT 2016
t.p.northover added a comment.
I don't think that fixes it. I'm seeing discrepancies of up to 32 in the estimate even with that fix in place. Unfortunately this one's not playing well with llc so I don't have a reduced example, but I can help you reproduce it if you want (it's also from povray in the test-suite-externals).
Really, I think this shows just how misguided it is to try and use estimateStackSize in this way: neither of us really understands what it's doing and even if *we* did, future readers won't.
Cheers.
Tim.
http://reviews.llvm.org/D18619
More information about the llvm-commits
mailing list