[PATCH] D12098: [WinEH] Calculate state numbers for the new EH representation
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 18 11:48:08 PDT 2015
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: include/llvm/CodeGen/WinEHFuncInfo.h:125
@@ -124,3 +124,3 @@
int CatchObjRecoverIdx;
- Function *Handler;
+ Value *Handler;
};
----------------
This guy should be const. I'm reasonably confident that the const cascading will stop there, since all we use it for is to get an MCSymbol.
================
Comment at: lib/CodeGen/WinEHPrepare.cpp:2611
@@ +2610,3 @@
+ }
+ HT.Handler = const_cast<BasicBlock *>(CPI->getParent());
+ // FIXME: Pass CPI->getArgOperand(1).
----------------
We should kill this const_cast too.
http://reviews.llvm.org/D12098
More information about the llvm-commits
mailing list