[PATCH] D27227: Move most EH from MachineModuleInfo to MachineFunction

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 16:37:04 PST 2016


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm Thanks, I think we just never questioned why that stuff lived on MMI when doing all that funclet work.



================
Comment at: include/llvm/CodeGen/MachineFunction.h:30
 #include "llvm/MC/MCDwarf.h"
+#include "llvm/MC/MCSymbol.h"
 #include "llvm/Support/Allocator.h"
----------------
We can probably forward declare class MCSymbol.


================
Comment at: include/llvm/CodeGen/MachineModuleInfo.h:235
   /// none.
   unsigned getCurrentCallSite() { return CurCallSite; }
 
----------------
Mmmm, delicious SjLj state.


Repository:
  rL LLVM

https://reviews.llvm.org/D27227





More information about the llvm-commits mailing list