[PATCH] Remap frame variables used in outlined C++ exception handlers

Andy Kaylor andrew.kaylor at intel.com
Mon Feb 23 11:36:38 PST 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/CodeGen/WinEHPrepare.cpp:600-602
@@ +599,5 @@
+  if (AV) {
+    AllocaInst *NewAlloca = Builder.CreateAlloca(
+        AV->getAllocatedType(), AV->getArraySize(), AV->getName());
+    FrameVarInfo[AV].Allocas.push_back(NewAlloca);
+    return NewAlloca;
----------------
majnemer wrote:
> Would it be wrong to call `clone` on `AV`?   This doesn't handle `inalloca` correctly.
I'll look into that for the next revision.  It sounds like a good idea.

http://reviews.llvm.org/D7770

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list