AArch64: Safely handle the incoming sret call argument.
authorAndrew Trick <atrick@apple.com>
Mon, 16 Feb 2015 18:10:47 +0000 (18:10 +0000)
committerAndrew Trick <atrick@apple.com>
Mon, 16 Feb 2015 18:10:47 +0000 (18:10 +0000)
commit05938a548139554dbc5bd44c9837da5f412ab85c
treecd7c7d2f8b602da68cda1fdbf44707a3c4c3c5c8
parentc64150b8f3981835a23cf82377a243c163c92f47
AArch64: Safely handle the incoming sret call argument.

This adds a safe interface to the machine independent InputArg struct
for accessing the index of the original (IR-level) argument. When a
non-native return type is lowered, we generate the hidden
machine-level sret argument on-the-fly. Before this fix, we were
representing this argument as OrigArgIndex == 0, which is an outright
lie. In particular this crashed in the AArch64 backend where we
actually try to access the type of the original argument.

Now we use a sentinel value for machine arguments that have no
original argument index. AArch64, ARM, Mips, and PPC now check for this
case before accessing the original argument.

Fixes <rdar://19792160> Null pointer assertion in AArch64TargetLowering

llvm-svn: 229413
llvm/include/llvm/Target/TargetCallingConv.h
llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
llvm/lib/Target/ARM/ARMISelLowering.cpp
llvm/lib/Target/Mips/MipsCCState.cpp
llvm/lib/Target/Mips/MipsISelLowering.cpp
llvm/lib/Target/PowerPC/PPCISelLowering.cpp
llvm/lib/Target/R600/R600ISelLowering.cpp
llvm/lib/Target/R600/SIISelLowering.cpp
llvm/test/CodeGen/AArch64/implicit-sret.ll [new file with mode: 0644]