diff options
author | Hal Finkel <hfinkel@anl.gov> | 2016-09-01 10:28:47 +0000 |
---|---|---|
committer | Hal Finkel <hfinkel@anl.gov> | 2016-09-01 10:28:47 +0000 |
commit | 77579f5cd8c11cce70ca3f20593cf6cdb6603ea9 (patch) | |
tree | 37bd7362f101de1996b434d25aac0544f9458ad7 /test/CodeGen/PowerPC/eh-dwarf-cfa.ll | |
parent | 81276830eea84f1b49543a236aa3d45c1e7dcd3d (diff) |
Add ISD::EH_DWARF_CFA, simplify @llvm.eh.dwarf.cfa on Mips, fix on PowerPC
LLVM has an @llvm.eh.dwarf.cfa intrinsic, used to lower the GCC-compatible
__builtin_dwarf_cfa() builtin. As pointed out in PR26761, this is currently
broken on PowerPC (and likely on ARM as well). Currently, @llvm.eh.dwarf.cfa is
lowered using:
ADD(FRAMEADDR, FRAME_TO_ARGS_OFFSET)
where FRAME_TO_ARGS_OFFSET defaults to the constant zero. On x86,
FRAME_TO_ARGS_OFFSET is lowered to 2*SlotSize. This setup, however, does not
work for PowerPC. Because of the way that the stack layout works, the canonical
frame address is not exactly (FRAMEADDR + FRAME_TO_ARGS_OFFSET) on PowerPC
(there is a lower save-area offset as well), so it is not just a matter of
implementing FRAME_TO_ARGS_OFFSET for PowerPC (unless we redefine its
semantics -- We can do that, since it is currently used only for
@llvm.eh.dwarf.cfa lowering, but the better to directly lower the CFA construct
itself (since it can be easily represented as a fixed-offset FrameIndex)). Mips
currently does this, but by using a custom lowering for ADD that specifically
recognizes the (FRAMEADDR, FRAME_TO_ARGS_OFFSET) pattern.
This change introduces a ISD::EH_DWARF_CFA node, which by default expands using
the existing logic, but can be directly lowered by the target. Mips is updated
to use this method (which simplifies its implementation, and I suspect makes it
more robust), and updates PowerPC to do the same.
Fixes PR26761.
Differential Revision: https://reviews.llvm.org/D24038
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@280350 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGen/PowerPC/eh-dwarf-cfa.ll')
-rw-r--r-- | test/CodeGen/PowerPC/eh-dwarf-cfa.ll | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/test/CodeGen/PowerPC/eh-dwarf-cfa.ll b/test/CodeGen/PowerPC/eh-dwarf-cfa.ll new file mode 100644 index 00000000000..b8f6a596823 --- /dev/null +++ b/test/CodeGen/PowerPC/eh-dwarf-cfa.ll @@ -0,0 +1,28 @@ +; RUN: llc < %s | FileCheck %s +target datalayout = "e-m:e-i64:64-n32:64" +target triple = "powerpc64le-unknown-linux-gnu" + +define void @_Z1fv() #0 { +entry: + %0 = call i8* @llvm.eh.dwarf.cfa(i32 0) + call void @_Z1gPv(i8* %0) + ret void + +; CHECK-LABEL: @_Z1fv +; CHECK: stdu 1, -[[SS:[0-9]+]](1) +; CHECK: .cfi_def_cfa_offset [[SS]] +; CHECK: mr 31, 1 +; CHECK: .cfi_def_cfa_register r31 +; CHECK: addi 3, 31, [[SS]] +; CHECK-NEXT: bl _Z1gPv +; CHECK: blr +} + +declare void @_Z1gPv(i8*) + +; Function Attrs: nounwind +declare i8* @llvm.eh.dwarf.cfa(i32) #1 + +attributes #0 = { "no-frame-pointer-elim"="true" "target-cpu"="ppc64le" } +attributes #1 = { nounwind } + |