[PATCH] Fix hardcoded stack probe space

Andrew H clangbugs.10.mdwxzxkl at spamgourmet.com
Wed Dec 24 01:02:54 PST 2014


Fixed handling of -stack-probe-size=0. In contrast to the stack alignment override, zero is a valid value for the stack probe size override (i.e. always generate stack probing calls, regardless of how many bytes the local variables occupy). Unfortunately that means a little bit more management overhead, as there is no magic value for the stack probe size that indicates that the users wants to use the default size. On the plus side, the driver does not need to know any internals, like that the default page size is 4096 bytes.

Added test cases.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6684

Files:
  include/llvm/CodeGen/CommandFlags.h
  include/llvm/Target/TargetOptions.h
  lib/Target/X86/X86FrameLowering.cpp
  lib/Target/X86/X86Subtarget.cpp
  lib/Target/X86/X86Subtarget.h
  lib/Target/X86/X86TargetMachine.cpp
  test/CodeGen/X86/stack-probe-size.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6684.17620.patch
Type: text/x-patch
Size: 11572 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141224/2e8dbf9c/attachment.bin>


More information about the llvm-commits mailing list