diff options
| author | Shoaib Meenai <smeenai@fb.com> | 2018-06-07 22:24:20 +0000 |
|---|---|---|
| committer | Shoaib Meenai <smeenai@fb.com> | 2018-06-07 22:24:20 +0000 |
| commit | 151f14ccdb30e8b10183c2a198d63c0ff3fb4308 (patch) | |
| tree | 06766d081979dd62dd658371fe7caaf4d2dc702d /clang/lib/Parse | |
| parent | aa46ed92786077ae0b779372d52a73085965c1c8 (diff) | |
| download | bcm5719-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.cpp | 15 |
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; |

