[PATCH] D52748: [WebAssembly] LSDA info generation

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 15:02:30 PDT 2018


aheejin added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1186
+              int Index = cast<ConstantInt>(IndexArg)->getZExtValue();
+              MF->setWasmLandingPadIndex(MBB, Index);
+              IntrFound = true;
----------------
rnk wrote:
> I see, so this intrinsic only exists to associate this landing pad index number with the catchpad. If we had some other way to staple the number onto catchpad, that would be nicer. This seems like an OK way to do this.
Do you think I can put the index number in a bundle argument to catchpad? Like, a named bundle operand "index".


Repository:
  rL LLVM

https://reviews.llvm.org/D52748





More information about the llvm-commits mailing list