[PATCH] D54614: [WebAssembly] Fix MCNullStreamer support
    Eugene Sharygin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Nov 16 14:17:06 PST 2018
    
    
  
eush added inline comments.
================
Comment at: test/CodeGen/WebAssembly/null-streamer.ll:11
+define i32 @f(i8* %p)  {
+  call void @llvm.wasm.throw(i32 0, i8* %p)
+  call void @g()
----------------
sbc100 wrote:
> Can you simplify this test, drop the trow and remove the exception arguments from llc?
Can I wonder why? A simpler test would only exercise changes in EmitFunctionBodyStart and wouldn't need any changes in EmitEndOfAsmFile.
Repository:
  rL LLVM
https://reviews.llvm.org/D54614
    
    
More information about the llvm-commits
mailing list