[PATCH] D20852: [AMDGPU] Remove exit-on-error in test (PR27761)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 6 13:37:54 PDT 2016
arsenm added inline comments.
================
Comment at: lib/Target/AMDGPU/AMDGPUISelLowering.cpp:692
@@ -691,2 +691,3 @@
DAG.getContext()->diagnose(NoDynamicAlloca);
- return SDValue();
+ auto Ops = {DAG.getIntPtrConstant(0, SDLoc()), Op.getOperand(0)};
+ return DAG.getMergeValues(Ops, SDLoc());
----------------
I would prefer not using getIntPtrConstant since I would like to remove it in general from LLVM. This would more preferrably be getConstant(-1, Op.getValueType())
================
Comment at: test/CodeGen/AMDGPU/promote-alloca-bitcast-function.ll:9
@@ -8,3 +8,3 @@
; CHECK: in function crash_call_constexpr_cast{{.*}}: unsupported call to function foo
define void @crash_call_constexpr_cast() #0 {
----------------
I think this can now have a check for the second function's error
http://reviews.llvm.org/D20852
More information about the llvm-commits
mailing list