[llvm-commits] [llvm] r108581 - /llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp
Eric Christopher
echristo at apple.com
Fri Jul 16 17:25:41 PDT 2010
Author: echristo
Date: Fri Jul 16 19:25:41 2010
New Revision: 108581
URL: http://llvm.org/viewvc/llvm-project?rev=108581&view=rev
Log:
Make comment a bit more clear as well as return statement since
needsStackRealignment is currently checking the can conditions as well.
Modified:
llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp
Modified: llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp?rev=108581&r1=108580&r2=108581&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86RegisterInfo.cpp Fri Jul 16 19:25:41 2010
@@ -462,12 +462,12 @@
// FIXME: Currently we don't support stack realignment for functions with
// variable-sized allocas.
- // FIXME: Temporary disable the error - it seems to be too conservative.
+ // FIXME: It's more complicated than this...
if (0 && requiresRealignment && MFI->hasVarSizedObjects())
report_fatal_error(
"Stack realignment in presense of dynamic allocas is not supported");
- return (requiresRealignment && !MFI->hasVarSizedObjects());
+ return requiresRealignment && canRealignStack(MF);
}
bool X86RegisterInfo::hasReservedCallFrame(MachineFunction &MF) const {
More information about the llvm-commits
mailing list