[PATCH] D49208: [WebAssembly] Update WebAssemblyLowerEmscriptenEHSjLj to support separate compilation.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 28 15:15:16 PDT 2018


sbc100 marked 3 inline comments as done.
sbc100 added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp:487
       return false;
+  }
 
----------------
dschuff wrote:
> jgravelle-google wrote:
> > Nit: unneeded braces
> Nit: no, those braces are needed IMHO :D
At least in lld the policy is not to include them this these cases.


Repository:
  rL LLVM

https://reviews.llvm.org/D49208





More information about the llvm-commits mailing list