[PATCH] D47564: [Parse] Use CapturedStmt for @finally on MSVC

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 31 13:34:00 PDT 2018


rnk added a comment.

Using CapturedStmt to do frontend outlining was the direction I suggested. I want to hear what @rsmith and @rjmccall think, though.



================
Comment at: lib/Parse/ParseObjc.cpp:2588
 
+      bool ShouldCapture = Actions.getASTContext()
+                               .getTargetInfo()
----------------
Parser has a `getTargetInfo()` method, so this can be shorter.


Repository:
  rC Clang

https://reviews.llvm.org/D47564





More information about the cfe-commits mailing list