[PATCH] D54070: [WebAssembly] Update test cases after main wrapper generation
    Sam Clegg via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Nov  5 09:33:06 PST 2018
    
    
  
sbc100 added a comment.
Hmm.. just to be clear, these tests are not failing on HEAD are they?
Why not just leave them as is?
Also, I've had some fallout from the change to enable the `main` wrappers by default so I think we should perhaps revert it.  Specifically the `test_mainenv` test in emscripten is failing because it expects envp to be set to null, not some undefined value (which happens to be non-null right now).
Repository:
  rL LLVM
https://reviews.llvm.org/D54070
    
    
More information about the llvm-commits
mailing list