diff options
| author | Than McIntosh <thanm@google.com> | 2017-09-27 19:34:00 +0000 |
|---|---|---|
| committer | Than McIntosh <thanm@google.com> | 2017-09-27 19:34:00 +0000 |
| commit | dee2cf67eafd0313a5f0f163b9cbe6c71228ae34 (patch) | |
| tree | d7d27d63cac4f03f7cfcf069d49e1400c46b97e4 /llvm/lib/CodeGen/PrologEpilogInserter.cpp | |
| parent | 93e33f8fb3dbc3d08ff5c99b184648f5e52265a0 (diff) | |
| download | bcm5719-llvm-dee2cf67eafd0313a5f0f163b9cbe6c71228ae34.tar.gz bcm5719-llvm-dee2cf67eafd0313a5f0f163b9cbe6c71228ae34.zip | |
[CodeGen] Emit necessary .note sections for -fsplit-stack
Summary:
According to https://gcc.gnu.org/wiki/SplitStacks, the linker expects a zero-sized .note.GNU-split-stack section if split-stack is used (and also .note.GNU-no-split-stack section if it also contains non-split-stack functions), so it can handle the cases where a split-stack function calls non-split-stack function.
This change adds the sections if needed.
Fixes PR #34670.
Reviewers: thanm, rnk, luqmana
Reviewed By: rnk
Subscribers: llvm-commits
Patch by Cherry Zhang <cherryyz@google.com>
Differential Revision: https://reviews.llvm.org/D38051
llvm-svn: 314335
Diffstat (limited to 'llvm/lib/CodeGen/PrologEpilogInserter.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/PrologEpilogInserter.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/PrologEpilogInserter.cpp b/llvm/lib/CodeGen/PrologEpilogInserter.cpp index 6e442065ef7..032abb441dd 100644 --- a/llvm/lib/CodeGen/PrologEpilogInserter.cpp +++ b/llvm/lib/CodeGen/PrologEpilogInserter.cpp @@ -994,7 +994,11 @@ void PEI::insertPrologEpilogCode(MachineFunction &Fn) { if (Fn.shouldSplitStack()) { for (MachineBasicBlock *SaveBlock : SaveBlocks) TFI.adjustForSegmentedStacks(Fn, *SaveBlock); - } + // Record that there are split-stack functions, so we will emit a + // special section to tell the linker. + Fn.getMMI().setHasSplitStack(true); + } else + Fn.getMMI().setHasNosplitStack(true); // Emit additional code that is required to explicitly handle the stack in // HiPE native code (if needed) when loaded in the Erlang/OTP runtime. The |
