[PATCH] Outline functions for SEH with the MSVC environment

Reid Kleckner rnk at google.com
Fri Dec 12 15:50:16 PST 2014


Neat, so these tests pass? Seems like this approach is working! =D

The MSVCEHPrepare.cpp file has a bunch of Allman style braces, which should be fixed.

I feel like this is probably a good chunk of logic to commit, but we're a bit blocked on review. Long term, will need to do analysis to share regions of cleanup code in separate outlined functions rather than duplicating code.


REPOSITORY
  rL LLVM

================
Comment at: lib/Transforms/Utils/CloneFunction.cpp:327
@@ +326,3 @@
+                              = Director->handleInstruction(VMap, II, NewBB);
+      // If the stop handler says stop, we don't even want to stop everything,
+      // not just break out of the loop (which would cause the terminator to
----------------
This comment seems like it got mis-edited. :) Maybe "..., we want to stop everything. The cloning director is reponsible for inserting a terminator here."

================
Comment at: lib/Transforms/Utils/CloneFunction.cpp:383
@@ +382,3 @@
+                           = Director->handleInstruction(VMap, OldTI, NewBB);
+    // If the stop handler says stop, we don't even want to stop everything,
+    // not just break out of the loop (which would cause the terminator to
----------------
ditto

http://reviews.llvm.org/D6556

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






More information about the llvm-commits mailing list