[PATCH] D52748: [WebAssembly] LSDA info generation

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1186
+              int Index = cast<ConstantInt>(IndexArg)->getZExtValue();
+              MF->setWasmLandingPadIndex(MBB, Index);
+              IntrFound = true;
----------------
aheejin wrote:
> 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".
Operand bundles seem to only can be attached to `CallInst` and `InvokeInst`, so this is not gonna work.


Repository:
  rL LLVM

https://reviews.llvm.org/D52748