[PATCH] D15377: [WebAssembly] Reintroduce ARGUMENT moving logic
    JF Bastien via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Dec  9 06:41:51 PST 2015
    
    
  
jfb added a comment.
The order of the ARGUMENTs isn't changed by SelectionDAG?
This seems to fix the issue, but I'm not familiar enough with SelectionDAG to have an opinion about what it should be doing.
================
Comment at: lib/Target/WebAssembly/WebAssemblyArgumentMove.cpp:57
@@ +56,3 @@
+
+private:
+};
----------------
Dead.
================
Comment at: lib/Target/WebAssembly/WebAssemblyArgumentMove.cpp:82
@@ +81,3 @@
+        MI->getOpcode() != WebAssembly::ARGUMENT_F32 &&
+        MI->getOpcode() != WebAssembly::ARGUMENT_F64) {
+      InsertPt = MII;
----------------
Factor out `isArgument()`?
Repository:
  rL LLVM
http://reviews.llvm.org/D15377
    
    
More information about the llvm-commits
mailing list