summaryrefslogtreecommitdiffstats
path: root/clang/lib/Parse
diff options
context:
space:
mode:
authorShoaib Meenai <smeenai@fb.com>2018-06-07 22:24:20 +0000
committerShoaib Meenai <smeenai@fb.com>2018-06-07 22:24:20 +0000
commit151f14ccdb30e8b10183c2a198d63c0ff3fb4308 (patch)
tree06766d081979dd62dd658371fe7caaf4d2dc702d /clang/lib/Parse
parentaa46ed92786077ae0b779372d52a73085965c1c8 (diff)
downloadbcm5719-llvm-151f14ccdb30e8b10183c2a198d63c0ff3fb4308.tar.gz
bcm5719-llvm-151f14ccdb30e8b10183c2a198d63c0ff3fb4308.zip
Revert "[Parse] Use CapturedStmt for @finally on MSVC"
This reverts commit r334224. This is causing buildbot failures on Windows, presumably because some tests don't specify a triple. I'll test this on Windows locally and recommit with the tests fixed. llvm-svn: 334240
Diffstat (limited to 'clang/lib/Parse')
-rw-r--r--clang/lib/Parse/ParseObjc.cpp15
1 files changed, 1 insertions, 14 deletions
diff --git a/clang/lib/Parse/ParseObjc.cpp b/clang/lib/Parse/ParseObjc.cpp
index ff33d5fb96b..6ca0ad855f5 100644
--- a/clang/lib/Parse/ParseObjc.cpp
+++ b/clang/lib/Parse/ParseObjc.cpp
@@ -2585,26 +2585,13 @@ StmtResult Parser::ParseObjCTryStmt(SourceLocation atLoc) {
ParseScope FinallyScope(this,
Scope::DeclScope | Scope::CompoundStmtScope);
- bool ShouldCapture =
- getTargetInfo().getTriple().isWindowsMSVCEnvironment();
- if (ShouldCapture)
- Actions.ActOnCapturedRegionStart(Tok.getLocation(), getCurScope(),
- CR_ObjCAtFinally, 1);
-
StmtResult FinallyBody(true);
if (Tok.is(tok::l_brace))
FinallyBody = ParseCompoundStatementBody();
else
Diag(Tok, diag::err_expected) << tok::l_brace;
-
- if (FinallyBody.isInvalid()) {
+ if (FinallyBody.isInvalid())
FinallyBody = Actions.ActOnNullStmt(Tok.getLocation());
- if (ShouldCapture)
- Actions.ActOnCapturedRegionError();
- } else if (ShouldCapture) {
- FinallyBody = Actions.ActOnCapturedRegionEnd(FinallyBody.get());
- }
-
FinallyStmt = Actions.ActOnObjCAtFinallyStmt(AtCatchFinallyLoc,
FinallyBody.get());
catch_or_finally_seen = true;
OpenPOWER on IntegriCloud