[PATCH] D71337: [VE,#0] 64bit data for SjLj

[Simon Moll via Phabricator via llvm-commits]

simoll planned changes to this revision.
simoll marked an inline comment as done.
simoll added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetPassConfig.cpp:698-702
+    if (TM->getTargetTriple().isVE())
+      addPass(createSjLjEHPreparePass(true));
+    else
+      addPass(createSjLjEHPreparePass());
     LLVM_FALLTHROUGH;
----------------
arsenm wrote:
> I would expect this to be a target hook of some kind the pass checks, not a pass parameter controlled here. Checking a target triple here is probably not kosher
Ok. How about querying the SjLj data size from TTI (eg `TTI::getSjLjDataSize()`)?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71337/new/

https://reviews.llvm.org/D71337