[llvm] r243770 - WebAssembly: handle unused function arguments.
JF Bastien
jfb at google.com
Fri Jul 31 11:13:27 PDT 2015
Author: jfb
Date: Fri Jul 31 13:13:27 2015
New Revision: 243770
URL: http://llvm.org/viewvc/llvm-project?rev=243770&view=rev
Log:
WebAssembly: handle unused function arguments.
Subscribers: llvm-commits, sunfish, jfb
Differential Revision: http://reviews.llvm.org/D11684
Added:
llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll
Modified:
llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp?rev=243770&r1=243769&r2=243770&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp Fri Jul 31 13:13:27 2015
@@ -191,11 +191,13 @@ SDValue WebAssemblyTargetLowering::Lower
fail(DL, DAG, "WebAssembly hasn't implemented split arguments");
if (In.VT != MVT::i32)
fail(DL, DAG, "WebAssembly hasn't implemented non-i32 arguments");
- if (!In.Used)
- fail(DL, DAG, "WebAssembly hasn't implemented unused arguments");
// FIXME Do something with In.getOrigAlign()?
- InVals.push_back(DAG.getNode(WebAssemblyISD::ARGUMENT, DL, In.VT,
- DAG.getTargetConstant(ArgNo++, DL, MVT::i32)));
+ InVals.push_back(
+ In.Used
+ ? DAG.getNode(WebAssemblyISD::ARGUMENT, DL, In.VT,
+ DAG.getTargetConstant(ArgNo, DL, MVT::i32))
+ : DAG.getNode(ISD::UNDEF, DL, In.VT));
+ ++ArgNo;
}
return Chain;
Added: llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll?rev=243770&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll (added)
+++ llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll Fri Jul 31 13:13:27 2015
@@ -0,0 +1,20 @@
+; RUN: llc < %s -asm-verbose=false | FileCheck %s
+
+; Make sure that argument offsets are correct even if some arguments are unused.
+
+target datalayout = "e-p:32:32-i64:64-v128:8:128-n32:64-S128"
+target triple = "wasm32-unknown-unknown"
+
+; CHECK-LABEL: unused_first:
+; CHECK-NEXT: (setlocal @0 (argument 1))
+; CHECK-NEXT: (RETURN @0)
+define i32 @unused_first(i32 %x, i32 %y) {
+ ret i32 %y
+}
+
+; CHECK-LABEL: unused_second:
+; CHECK-NEXT: (setlocal @0 (argument 0))
+; CHECK-NEXT: (RETURN @0)
+define i32 @unused_second(i32 %x, i32 %y) {
+ ret i32 %x
+}
More information about the llvm-commits
mailing list