[PATCH] D44090: [WebAssembly] Support instruction selection for catching exceptions

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 10 17:13:52 PDT 2018


dschuff added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:1367
   MachineBasicBlock *CatchPadMBB = FuncInfo.MBB;
   // In MSVC C++ and CoreCLR, catchblocks are funclets and need prologues.
+  if (IsMSVCCXX || IsCoreCLR || IsWasmCXX)
----------------
Based on this comment  I assume we are marking 'IsEHFuncletEntry' for a different purpose than MSVC (since we don't need a function prologue)? In that case I guess this comment should be updated. Likewise below. (what do we use it for?)


Repository:
  rL LLVM

https://reviews.llvm.org/D44090





More information about the llvm-commits mailing list