diff --git a/clang/include/clang/AST/Stmt.h b/clang/include/clang/AST/Stmt.h --- a/clang/include/clang/AST/Stmt.h +++ b/clang/include/clang/AST/Stmt.h @@ -1743,6 +1743,7 @@ class LabelStmt : public ValueStmt { LabelDecl *TheDecl; Stmt *SubStmt; + bool SideEntry = false; public: /// Build a label statement. @@ -1778,6 +1779,9 @@ static bool classof(const Stmt *T) { return T->getStmtClass() == LabelStmtClass; } + + bool isSideEntry() const { return SideEntry; } + void setSideEntry(bool SE) { SideEntry = SE; } }; /// Represents an attribute applied to a statement. diff --git a/clang/include/clang/Basic/LangOptions.def b/clang/include/clang/Basic/LangOptions.def --- a/clang/include/clang/Basic/LangOptions.def +++ b/clang/include/clang/Basic/LangOptions.def @@ -128,6 +128,7 @@ LANGOPT(Exceptions , 1, 0, "exception handling") LANGOPT(ObjCExceptions , 1, 0, "Objective-C exceptions") LANGOPT(CXXExceptions , 1, 0, "C++ exceptions") +LANGOPT(EHAsynch , 1, 0, "C/C++ EH Asynch exceptions") LANGOPT(DWARFExceptions , 1, 0, "dwarf exception handling") LANGOPT(SjLjExceptions , 1, 0, "setjmp-longjump exception handling") LANGOPT(SEHExceptions , 1, 0, "SEH .xdata exception handling") diff --git a/clang/include/clang/Driver/Options.td b/clang/include/clang/Driver/Options.td --- a/clang/include/clang/Driver/Options.td +++ b/clang/include/clang/Driver/Options.td @@ -883,6 +883,8 @@ def fcreate_profile : Flag<["-"], "fcreate-profile">, Group; def fcxx_exceptions: Flag<["-"], "fcxx-exceptions">, Group, HelpText<"Enable C++ exceptions">, Flags<[CC1Option]>; +def fasync_exceptions: Flag<["-"], "fasync-exceptions">, Group, + HelpText<"Enable EH Asynchronous exceptions">, Flags<[CC1Option]>; def fcxx_modules : Flag <["-"], "fcxx-modules">, Group, Flags<[DriverOption]>; def fdebug_pass_arguments : Flag<["-"], "fdebug-pass-arguments">, Group; @@ -1526,6 +1528,7 @@ Flags<[CC1Option]>, HelpText<"Disable creation of CodeFoundation-type constant strings">; def fno_cxx_exceptions: Flag<["-"], "fno-cxx-exceptions">, Group; +def fno_fasync_exceptions: Flag<["-"], "fno-fasync-exceptions">, Group; def fno_cxx_modules : Flag <["-"], "fno-cxx-modules">, Group, Flags<[DriverOption]>; def fno_diagnostics_fixit_info : Flag<["-"], "fno-diagnostics-fixit-info">, Group, diff --git a/clang/lib/AST/JSONNodeDumper.cpp b/clang/lib/AST/JSONNodeDumper.cpp --- a/clang/lib/AST/JSONNodeDumper.cpp +++ b/clang/lib/AST/JSONNodeDumper.cpp @@ -1448,6 +1448,7 @@ void JSONNodeDumper::VisitLabelStmt(const LabelStmt *LS) { JOS.attribute("name", LS->getName()); JOS.attribute("declId", createPointerRepresentation(LS->getDecl())); + attributeOnlyIfTrue("sideEntry", LS->isSideEntry()); } void JSONNodeDumper::VisitGotoStmt(const GotoStmt *GS) { JOS.attribute("targetLabelDeclId", diff --git a/clang/lib/AST/TextNodeDumper.cpp b/clang/lib/AST/TextNodeDumper.cpp --- a/clang/lib/AST/TextNodeDumper.cpp +++ b/clang/lib/AST/TextNodeDumper.cpp @@ -695,6 +695,8 @@ void TextNodeDumper::VisitLabelStmt(const LabelStmt *Node) { OS << " '" << Node->getName() << "'"; + if (Node->isSideEntry()) + OS << " side_entry"; } void TextNodeDumper::VisitGotoStmt(const GotoStmt *Node) { diff --git a/clang/lib/CodeGen/CGCleanup.cpp b/clang/lib/CodeGen/CGCleanup.cpp --- a/clang/lib/CodeGen/CGCleanup.cpp +++ b/clang/lib/CodeGen/CGCleanup.cpp @@ -196,6 +196,11 @@ if (IsLifetimeMarker) Scope->setLifetimeMarker(); + // With Windows -EHa, Invoke llvm.seh.scope.begin() for EHCleanup + if (CGF->getLangOpts().EHAsynch && IsEHCleanup && + CGF->getTarget().getCXXABI().isMicrosoft()) + CGF->EmitSehCppScopeBegin(); + return Scope->getCleanupBuffer(); } @@ -761,14 +766,31 @@ if (Scope.isEHCleanup()) cleanupFlags.setIsEHCleanupKind(); + // Under -EHa, invoke seh.scope.end() to mark scope end before dtor + bool IsEHa = getLangOpts().EHAsynch && !Scope.isLifetimeMarker(); + const EHPersonality& Personality = EHPersonality::get(*this); if (!RequiresNormalCleanup) { + // Mark CPP scope end for passed-by-value Arg temp + // per Windows ABI which is "normally" Cleanup in callee + if (IsEHa && getInvokeDest()) { + if (Personality.isMSVCXXPersonality()) + EmitSehCppScopeEnd(); + } destroyOptimisticNormalEntry(*this, Scope); EHStack.popCleanup(); } else { // If we have a fallthrough and no other need for the cleanup, // emit it directly. - if (HasFallthrough && !HasPrebranchedFallthrough && - !HasFixups && !HasExistingBranches) { + if (HasFallthrough && !HasPrebranchedFallthrough && !HasFixups && + !HasExistingBranches) { + + // mark SEH scope end for fall-through flow + if (IsEHa && getInvokeDest()) { + if (Personality.isMSVCXXPersonality()) + EmitSehCppScopeEnd(); + else + EmitSehTryScopeEnd(); + } destroyOptimisticNormalEntry(*this, Scope); EHStack.popCleanup(); @@ -803,6 +825,14 @@ // should already be branched to it. EmitBlock(NormalEntry); + // intercept normal cleanup to mark SEH scope end + if (IsEHa) { + if (Personality.isMSVCXXPersonality()) + EmitSehCppScopeEnd(); + else + EmitSehTryScopeEnd(); + } + // III. Figure out where we're going and build the cleanup // epilogue. @@ -1250,11 +1280,17 @@ // to the current RunCleanupsScope. if (C == EHStack.stable_begin() && CurrentCleanupScopeDepth.strictlyEncloses(C)) { - // If it's a normal cleanup, we need to pretend that the - // fallthrough is unreachable. - CGBuilderTy::InsertPoint SavedIP = Builder.saveAndClearIP(); - PopCleanupBlock(); - Builder.restoreIP(SavedIP); + // Per comment below, checking EHAsynch is not really necessary + // it's there to assure zero-impact w/o EHAsynch option + if (!Scope.isNormalCleanup() && getLangOpts().EHAsynch) { + PopCleanupBlock(); + } else { + // If it's a normal cleanup, we need to pretend that the + // fallthrough is unreachable. + CGBuilderTy::InsertPoint SavedIP = Builder.saveAndClearIP(); + PopCleanupBlock(); + Builder.restoreIP(SavedIP); + } return; } @@ -1278,3 +1314,60 @@ pushDestroy(NormalAndEHCleanup, Ptr, TempType, destroyCXXObject, /*useEHCleanup*/ true); } + +// Need to set "funclet" in OperandBundle properly for noThrow +// intrinsic (see CGCall.cpp) +static void EmitSehScope(CodeGenFunction &CGF, + llvm::FunctionCallee &SehCppScope) { + llvm::BasicBlock *InvokeDest = CGF.getInvokeDest(); + llvm::BasicBlock *BB = CGF.Builder.GetInsertBlock(); + assert(BB && InvokeDest); + llvm::BasicBlock *Cont = CGF.createBasicBlock("invoke.cont"); + SmallVector BundleList = + CGF.getBundlesForFunclet(SehCppScope.getCallee()); + if (CGF.CurrentFuncletPad) + BundleList.emplace_back("funclet", CGF.CurrentFuncletPad); + CGF.Builder.CreateInvoke(SehCppScope, Cont, InvokeDest, None, BundleList); + CGF.EmitBlock(Cont); +} + +// Invoke a llvm.seh.scope.begin at the beginning of a CPP scope for -EHa +void CodeGenFunction::EmitSehCppScopeBegin() { + assert(getLangOpts().EHAsynch); + llvm::FunctionType *FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + llvm::FunctionCallee SehCppScope = + CGM.CreateRuntimeFunction(FTy, "llvm.seh.scope.begin"); + EmitSehScope(*this, SehCppScope); +} + +// Invoke a llvm.seh.scope.end at the end of a CPP scope for -EHa +// llvm.seh.scope.end is emitted before popCleanup, so it's "invoked" +void CodeGenFunction::EmitSehCppScopeEnd() { + assert(getLangOpts().EHAsynch); + llvm::FunctionType *FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + llvm::FunctionCallee SehCppScope = + CGM.CreateRuntimeFunction(FTy, "llvm.seh.scope.end"); + EmitSehScope(*this, SehCppScope); +} + +// Invoke a llvm.seh.try.begin at the beginning of a SEH scope for -EHa +void CodeGenFunction::EmitSehTryScopeBegin() { + assert(getLangOpts().EHAsynch); + llvm::FunctionType* FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + llvm::FunctionCallee SehCppScope = + CGM.CreateRuntimeFunction(FTy, "llvm.seh.try.begin"); + EmitSehScope(*this, SehCppScope); +} + +// Invoke a llvm.seh.try.end at the end of a SEH scope for -EHa +void CodeGenFunction::EmitSehTryScopeEnd() { + assert(getLangOpts().EHAsynch); + llvm::FunctionType* FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + llvm::FunctionCallee SehCppScope = + CGM.CreateRuntimeFunction(FTy, "llvm.seh.try.end"); + EmitSehScope(*this, SehCppScope); +} \ No newline at end of file diff --git a/clang/lib/CodeGen/CGException.cpp b/clang/lib/CodeGen/CGException.cpp --- a/clang/lib/CodeGen/CGException.cpp +++ b/clang/lib/CodeGen/CGException.cpp @@ -38,6 +38,18 @@ return CGM.CreateRuntimeFunction(FTy, "__cxa_free_exception"); } +static llvm::FunctionCallee getSehTryBeginFn(CodeGenModule &CGM) { + llvm::FunctionType *FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + return CGM.CreateRuntimeFunction(FTy, "llvm.seh.try.begin"); +} + +static llvm::FunctionCallee getSehTryEndFn(CodeGenModule &CGM) { + llvm::FunctionType *FTy = + llvm::FunctionType::get(CGM.VoidTy, /*isVarArg=*/false); + return CGM.CreateRuntimeFunction(FTy, "llvm.seh.try.end"); +} + static llvm::FunctionCallee getUnexpectedFn(CodeGenModule &CGM) { // void __cxa_call_unexpected(void *thrown_exception); @@ -462,7 +474,8 @@ ExceptionSpecificationType EST = Proto->getExceptionSpecType(); if (isNoexceptExceptionSpec(EST) && Proto->canThrow() == CT_Cannot) { // noexcept functions are simple terminate scopes. - EHStack.pushTerminate(); + if (!getLangOpts().EHAsynch) // -EHa: HW exception still can occur + EHStack.pushTerminate(); } else if (EST == EST_Dynamic || EST == EST_DynamicNone) { // TODO: Revisit exception specifications for the MS ABI. There is a way to // encode these in an object file but MSVC doesn't do anything with it. @@ -527,7 +540,7 @@ if (!FD) { // Check if CapturedDecl is nothrow and pop terminate scope for it. if (const CapturedDecl* CD = dyn_cast_or_null(D)) { - if (CD->isNothrow()) + if (CD->isNothrow() && !EHStack.empty()) EHStack.popTerminate(); } return; @@ -537,7 +550,8 @@ return; ExceptionSpecificationType EST = Proto->getExceptionSpecType(); - if (isNoexceptExceptionSpec(EST) && Proto->canThrow() == CT_Cannot) { + if (isNoexceptExceptionSpec(EST) && Proto->canThrow() == CT_Cannot && + !EHStack.empty() /* possible empty when under async exceptions */) { EHStack.popTerminate(); } else if (EST == EST_Dynamic || EST == EST_DynamicNone) { // TODO: Revisit exception specifications for the MS ABI. There is a way to @@ -585,6 +599,10 @@ } else { // No exception decl indicates '...', a catch-all. CatchScope->setHandler(I, CGM.getCXXABI().getCatchAllTypeInfo(), Handler); + // Under async exceptions, catch(...) need to catch HW exception too + // Mark scope with SehTryBegin as a SEH __try scope + if (getLangOpts().EHAsynch) + EmitRuntimeCallOrInvoke(getSehTryBeginFn(CGM)); } } } @@ -706,7 +724,7 @@ // If exceptions are disabled/ignored and SEH is not in use, then there is no // invoke destination. SEH "works" even if exceptions are off. In practice, // this means that C++ destructors and other EH cleanups don't run, which is - // consistent with MSVC's behavior. + // consistent with MSVC's behavior, except in the presence of -EHa const LangOptions &LO = CGM.getLangOpts(); if (!LO.Exceptions || LO.IgnoreExceptions) { if (!LO.Borland && !LO.MicrosoftExt) @@ -1602,7 +1620,23 @@ JumpDest TryExit = getJumpDestInCurrentScope("__try.__leave"); SEHTryEpilogueStack.push_back(&TryExit); + + llvm::BasicBlock *TryBB = nullptr; + // IsEHa: emit an invoke to _seh_try_begin() runtime for -EHa + if (getLangOpts().EHAsynch) { + EmitRuntimeCallOrInvoke(getSehTryBeginFn(CGM)); + if (SEHTryEpilogueStack.size() == 1) // outermost only + TryBB = Builder.GetInsertBlock(); + } + EmitStmt(S.getTryBlock()); + + // Volatilize all blocks in Try, till current insert point + if (TryBB) { + llvm::SmallPtrSet Visited; + VolatilizeTryBlocks(TryBB, Visited); + } + SEHTryEpilogueStack.pop_back(); if (!TryExit.getBlock()->use_empty()) @@ -1613,6 +1647,38 @@ ExitSEHTryStmt(S); } +// Recursively walk through blocks in a _try +// and make all memory instructions volatile +void CodeGenFunction::VolatilizeTryBlocks( + llvm::BasicBlock *BB, llvm::SmallPtrSet &V) { + if (BB == SEHTryEpilogueStack.back()->getBlock() /* end of Try */ || + !V.insert(BB).second /* already visited */ || + !BB->getParent() /* not emitted */ || BB->empty()) + return; + + if (!BB->isEHPad()) { + for (llvm::BasicBlock::iterator J = BB->begin(), JE = BB->end(); J != JE; + ++J) { + if (isa(J)) { + auto LI = dyn_cast(J); + LI->setVolatile(true); + } else if (isa(J)) { + auto SI = dyn_cast(J); + SI->setVolatile(true); + } else if (isa(J)) { + auto *MCI = dyn_cast(J); + MCI->setVolatile(llvm::ConstantInt::get(Builder.getInt1Ty(), 1)); + } + } + } + const llvm::Instruction *TI = BB->getTerminator(); + if (TI) { + unsigned N = TI->getNumSuccessors(); + for (unsigned I = 0; I < N; I++) + VolatilizeTryBlocks(TI->getSuccessor(I), V); + } +} + namespace { struct PerformSEHFinally final : EHScopeStack::Cleanup { llvm::Function *OutlinedFinally; @@ -2050,6 +2116,12 @@ return; } + // IsEHa: emit an invoke _seh_try_end() to mark end of FT flow + if (getLangOpts().EHAsynch && Builder.GetInsertBlock()) { + llvm::FunctionCallee SehTryEnd = getSehTryEndFn(CGM); + EmitRuntimeCallOrInvoke(SehTryEnd); + } + // Otherwise, we must have an __except block. const SEHExceptStmt *Except = S.getExceptHandler(); assert(Except && "__try must have __finally xor __except"); diff --git a/clang/lib/CodeGen/CGStmt.cpp b/clang/lib/CodeGen/CGStmt.cpp --- a/clang/lib/CodeGen/CGStmt.cpp +++ b/clang/lib/CodeGen/CGStmt.cpp @@ -604,6 +604,11 @@ void CodeGenFunction::EmitLabelStmt(const LabelStmt &S) { EmitLabel(S.getDecl()); + + // IsEHa - emit eha.scope.begin if it's a side entry of a scope + if (getLangOpts().EHAsynch && S.isSideEntry()) + EmitSehCppScopeBegin(); + EmitStmt(S.getSubStmt()); } diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -2687,6 +2687,11 @@ void EmitCXXTemporary(const CXXTemporary *Temporary, QualType TempType, Address Ptr); + void EmitSehCppScopeBegin(); + void EmitSehCppScopeEnd(); + void EmitSehTryScopeBegin(); + void EmitSehTryScopeEnd(); + llvm::Value *EmitLifetimeStart(uint64_t Size, llvm::Value *Addr); void EmitLifetimeEnd(llvm::Value *Size, llvm::Value *Addr); @@ -3037,6 +3042,8 @@ void EmitSEHLeaveStmt(const SEHLeaveStmt &S); void EnterSEHTryStmt(const SEHTryStmt &S); void ExitSEHTryStmt(const SEHTryStmt &S); + void VolatilizeTryBlocks(llvm::BasicBlock *BB, + llvm::SmallPtrSet &V); void pushSEHCleanup(CleanupKind kind, llvm::Function *FinallyFunc); diff --git a/clang/lib/CodeGen/CodeGenFunction.cpp b/clang/lib/CodeGen/CodeGenFunction.cpp --- a/clang/lib/CodeGen/CodeGenFunction.cpp +++ b/clang/lib/CodeGen/CodeGenFunction.cpp @@ -70,6 +70,7 @@ CGM.getCodeGenOpts(), CGM.getLangOpts())) { if (!suppressNewContext) CGM.getCXXABI().getMangleContext().startNewFunction(); + EHStack.setCGF(this); llvm::FastMathFlags FMF; if (CGM.getLangOpts().FastMath) diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -597,6 +597,9 @@ llvm::DenormalMode::IEEE); } + if (LangOpts.EHAsynch) + getModule().addModuleFlag(llvm::Module::Warning, "eh-asynch", 1); + // Emit OpenCL specific module metadata: OpenCL/SPIR version. if (LangOpts.OpenCL) { EmitOpenCLMetadata(); diff --git a/clang/lib/CodeGen/EHScopeStack.h b/clang/lib/CodeGen/EHScopeStack.h --- a/clang/lib/CodeGen/EHScopeStack.h +++ b/clang/lib/CodeGen/EHScopeStack.h @@ -241,6 +241,9 @@ /// The innermost EH scope on the stack. stable_iterator InnermostEHScope; + /// The CGF this Stack belong to + CodeGenFunction* CGF; + /// The current set of branch fixups. A branch fixup is a jump to /// an as-yet unemitted label, i.e. a label for which we don't yet /// know the EH stack depth. Whenever we pop a cleanup, we have @@ -268,7 +271,7 @@ public: EHScopeStack() : StartOfBuffer(nullptr), EndOfBuffer(nullptr), StartOfData(nullptr), InnermostNormalCleanup(stable_end()), - InnermostEHScope(stable_end()) {} + InnermostEHScope(stable_end()), CGF(nullptr) {} ~EHScopeStack() { delete[] StartOfBuffer; } /// Push a lazily-created cleanup on the stack. @@ -316,6 +319,8 @@ std::memcpy(Buffer, Cleanup, Size); } + void setCGF(CodeGenFunction* inCGF) { CGF = inCGF; } + /// Pops a cleanup scope off the stack. This is private to CGCleanup.cpp. void popCleanup(); diff --git a/clang/lib/CodeGen/MicrosoftCXXABI.cpp b/clang/lib/CodeGen/MicrosoftCXXABI.cpp --- a/clang/lib/CodeGen/MicrosoftCXXABI.cpp +++ b/clang/lib/CodeGen/MicrosoftCXXABI.cpp @@ -131,7 +131,12 @@ /// MSVC needs an extra flag to indicate a catchall. CatchTypeInfo getCatchAllTypeInfo() override { - return CatchTypeInfo{nullptr, 0x40}; + // For -EHa catch(...) must handle HW exception + // Adjective = HT_IsStdDotDot (0x40), only catch C++ exceptions + if (getContext().getLangOpts().EHAsynch) + return CatchTypeInfo{nullptr, 0}; + else + return CatchTypeInfo{nullptr, 0x40}; } bool shouldTypeidBeNullChecked(bool IsDeref, QualType SrcRecordTy) override; diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -426,6 +426,7 @@ Args.ClaimAllArgs(options::OPT_fobjc_exceptions); Args.ClaimAllArgs(options::OPT_fno_objc_exceptions); Args.ClaimAllArgs(options::OPT_fcxx_exceptions); + Args.ClaimAllArgs(options::OPT_fasync_exceptions); Args.ClaimAllArgs(options::OPT_fno_cxx_exceptions); return; } @@ -434,6 +435,13 @@ bool EH = Args.hasFlag(options::OPT_fexceptions, options::OPT_fno_exceptions, false); + bool EHa = Args.hasFlag(options::OPT_fasync_exceptions, + options::OPT_fasync_exceptions, false); + if (EHa) { + CmdArgs.push_back("-fasync-exceptions"); + EH = true; + } + // Obj-C exceptions are enabled by default, regardless of -fexceptions. This // is not necessarily sensible, but follows GCC. if (types::isObjC(InputType) && @@ -6572,7 +6580,10 @@ if (types::isCXX(InputType)) CmdArgs.push_back("-fcxx-exceptions"); CmdArgs.push_back("-fexceptions"); + if (EH.Asynch) + CmdArgs.push_back("-fasync-exceptions"); } + if (types::isCXX(InputType) && EH.Synch && EH.NoUnwindC) CmdArgs.push_back("-fexternc-nounwind"); diff --git a/clang/lib/Frontend/CompilerInvocation.cpp b/clang/lib/Frontend/CompilerInvocation.cpp --- a/clang/lib/Frontend/CompilerInvocation.cpp +++ b/clang/lib/Frontend/CompilerInvocation.cpp @@ -2783,6 +2783,7 @@ Opts.IgnoreExceptions = Args.hasArg(OPT_fignore_exceptions); Opts.ObjCExceptions = Args.hasArg(OPT_fobjc_exceptions); Opts.CXXExceptions = Args.hasArg(OPT_fcxx_exceptions); + Opts.EHAsynch = Args.hasArg(OPT_fasync_exceptions); // -ffixed-point Opts.FixedPoint = diff --git a/clang/lib/Sema/JumpDiagnostics.cpp b/clang/lib/Sema/JumpDiagnostics.cpp --- a/clang/lib/Sema/JumpDiagnostics.cpp +++ b/clang/lib/Sema/JumpDiagnostics.cpp @@ -930,6 +930,9 @@ if (!ToScopesWarning.empty()) { S.Diag(DiagLoc, JumpDiagWarning); NoteJumpIntoScopes(ToScopesWarning); + assert(isa(To)); + LabelStmt *Label = cast(To); + Label->setSideEntry(true); } // Handle errors. diff --git a/clang/lib/Serialization/ASTReaderStmt.cpp b/clang/lib/Serialization/ASTReaderStmt.cpp --- a/clang/lib/Serialization/ASTReaderStmt.cpp +++ b/clang/lib/Serialization/ASTReaderStmt.cpp @@ -185,11 +185,13 @@ void ASTStmtReader::VisitLabelStmt(LabelStmt *S) { VisitStmt(S); + bool IsSideEntry = Record.readInt(); auto *LD = readDeclAs(); LD->setStmt(S); S->setDecl(LD); S->setSubStmt(Record.readSubStmt()); S->setIdentLoc(readSourceLocation()); + S->setSideEntry(IsSideEntry); } void ASTStmtReader::VisitAttributedStmt(AttributedStmt *S) { diff --git a/clang/lib/Serialization/ASTWriterStmt.cpp b/clang/lib/Serialization/ASTWriterStmt.cpp --- a/clang/lib/Serialization/ASTWriterStmt.cpp +++ b/clang/lib/Serialization/ASTWriterStmt.cpp @@ -115,6 +115,7 @@ void ASTStmtWriter::VisitLabelStmt(LabelStmt *S) { VisitStmt(S); + Record.push_back(S->isSideEntry()); Record.AddDeclRef(S->getDecl()); Record.AddStmt(S->getSubStmt()); Record.AddSourceLocation(S->getIdentLoc()); diff --git a/clang/test/CodeGen/windows-seh-EHa-CppCatchDotDotDot.cpp b/clang/test/CodeGen/windows-seh-EHa-CppCatchDotDotDot.cpp new file mode 100644 --- /dev/null +++ b/clang/test/CodeGen/windows-seh-EHa-CppCatchDotDotDot.cpp @@ -0,0 +1,58 @@ +// RUN: %clang_cc1 -triple x86_64-windows -fasync-exceptions -fcxx-exceptions -fexceptions -fms-extensions -x c++ -Wno-implicit-function-declaration -S -emit-llvm %s -o - | FileCheck %s + +// CHECK: define dso_local void @"?crash@@YAXH@Z +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() + +// CHECK: %[[dst:[0-9-]+]] = catchswitch within none [label %catch] unwind to caller +// CHECK: %[[dst1:[0-9-]+]] = catchpad within %[[dst]] [i8* null, i32 0, i8* null] +// CHECK: "funclet"(token %[[dst1]]) + +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: %[[src:[0-9-]+]] = load volatile i32, i32* %i +// CHECK-NEXT: invoke void @"?crash@@YAXH@Z"(i32 %[[src]]) +// CHECK: invoke void @llvm.seh.try.end() + +// ***************************************************************************** +// Abstract: Test CPP catch(...) under SEH -EHa option + +void printf(...); +int volatile *NullPtr = 0; +void foo() { + *NullPtr = 0; +} +int *pt1, *pt2, *pt3; +int g; +void crash(int i) { + g = i; + try { + struct A { + A() { + printf(" in A ctor \n"); + if (g == 0) + *NullPtr = 0; + } + ~A() { + printf(" in A dtor \n"); + } + } ObjA; + if (i == 1) + *NullPtr = 0; + } catch (...) { + printf(" in catch(...) funclet \n"); + if (i == 1) + throw(i); + } +} + +int main() { + for (int i = 0; i < 2; i++) { + __try { + crash(i); + } __except (1) { + printf(" Test CPP unwind: in except handler i = %d \n", i); + } + } + return 0; +} diff --git a/clang/test/CodeGen/windows-seh-EHa-CppCondiTemps.cpp b/clang/test/CodeGen/windows-seh-EHa-CppCondiTemps.cpp new file mode 100644 --- /dev/null +++ b/clang/test/CodeGen/windows-seh-EHa-CppCondiTemps.cpp @@ -0,0 +1,129 @@ +// RUN: %clang_cc1 -triple x86_64-windows -fasync-exceptions -fcxx-exceptions -fexceptions -fms-extensions -x c++ -Wno-implicit-function-declaration -S -emit-llvm %s -o - | FileCheck %s + +// CHECK: define dso_local i32 @"?bar@@YAHHVB1@@VB2@@@Z" +// CHECK: %coerce.dive1 = getelementptr inbounds %class.B2 +// CHECK: %coerce.dive2 = getelementptr inbounds %class.B1 +// ----- scope begin of two passed-by-value temps +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B1@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B2@@QEAA@XZ" + +// CHECK: define linkonce_odr dso_local void @"??1B2@@QEAA@XZ" +// CHECK: %this.addr = alloca %class.B2* +// ----- B1 scope begin without base ctor +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B1@@QEAA@XZ" + +// CHECK: define dso_local void @"?goo@@YA?AVB1@@H@Z" +// CHECK: call %class.B2* @"??0B2@@QEAA@XZ"(%class.B2* %b2ingoo) +// CHECK: invoke void @llvm.seh.scope.begin() +// check: call void @llvm.memcpy +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B2@@QEAA@XZ"(%class.B2* %b2ingoo) + +// CHECK: define linkonce_odr dso_local %class.B2* @"??0B2@@QEAA@XZ" +// CHECK: call %class.B1* @"??0B1@@QEAA@XZ"(%class.B1* +// ----- scope begin of base ctor +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() +// ----- B1 scope end without base dtor + +// **************************************************************************** +// Abstract: Test CPP Conditional-Expr & ABI Temps under SEH -EHa option + +void printf(...); + +int xxxx = 0; +int* ptr; + +int foo(int a) +{ + return xxxx + a; +} + +class B1 { +public: + int data = 90; + B1() { foo(data + 111); } + ~B1() { printf("in B1 Dtor \n"); } +}; +class B2 : public B1 { +public: + B2() { foo(data + 222); } + ~B2() { printf("in B2 Dtor \n");; } +}; +class B3 : public B2 { +public: + B3() { foo(data + 333); } + ~B3() { printf("in B3 Dtor \n");; } +}; + +int bar(int j, class B1 b1Bar, class B2 b2Bar) +{ + int ww; + if ( j > 0) + ww = b1Bar.data; + else + ww = b2Bar.data; + return ww + *ptr; +} + +class B1 goo(int w) +{ + class B2 b2ingoo; + b2ingoo.data += w; + return b2ingoo; +} + +// CHECK: define dso_local i32 @main() +// CHECK: invoke void @llvm.seh.scope.begin() +// --- beginning of conditional temp test +// CHECK: invoke %class.B2* @"??0B2@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke %class.B3* @"??0B3@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B3@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: call void @"??1B2@@QEAA@XZ" +// ----- end of conditional temp test + +// ----- testing caller's passed-by-value temps +// setting scope in case exception occurs before the call +// check: invoke %class.B2* @"??0B2@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke %class.B1* @"??0B1@@QEAA@XZ" +// CHECK: invoke void @llvm.seh.scope.begin() +// ----- end of temps' scope right before callee +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: invoke i32 @"?bar@@YAHHVB1@@VB2@@@Z" + +// ----- testing caller's return-by-value temp +// scope begins right after callee which is the ctor of return temp +// CHECK: void @"?goo@@YA?AVB1@@H@Z" +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() + +int main() { + class B3 b3inmain; + + // Test conditional ctor and dtor + int m = (xxxx > 1) ? B2().data + foo(99) : + B3().data + foo(88); + + // Test: passed-in by value + // Per Windows ABI, ctored by caller, dtored by callee + int i = bar(foo(0), B1(), B2()); + + // Test: returned by value + // Per Windows ABI, caller allocate a temp in stack, then ctored by callee, + // finally dtored in caller after consumed + class B1 b1fromgoo = goo(i); + + return m + b1fromgoo.data + b3inmain.data; +} \ No newline at end of file diff --git a/clang/test/CodeGen/windows-seh-EHa-CppDtors01.cpp b/clang/test/CodeGen/windows-seh-EHa-CppDtors01.cpp new file mode 100644 --- /dev/null +++ b/clang/test/CodeGen/windows-seh-EHa-CppDtors01.cpp @@ -0,0 +1,60 @@ +// RUN: %clang_cc1 -triple x86_64-windows fasync-exceptions -fcxx-exceptions -fexceptions -fms-extensions -x c++ -Wno-implicit-function-declaration -S -emit-llvm %s -o - | FileCheck %s + +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.begin() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: invoke void @llvm.seh.scope.end() +// CHECK: invoke void @llvm.seh.scope.end() + +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: %[[src:[0-9-]+]] = load volatile i32, i32* %i +// CHECK-NEXT: invoke void @"?crash@@YAXH@Z"(i32 %[[src]]) +// CHECK: invoke void @llvm.seh.try.end() + +// **************************************************************************** +// Abstract: Test CPP unwind Dtoring under SEH -EHa option + +void printf(...); +int volatile *NullPtr = 0; +void crash(int i) { + struct A { + ~A() { + printf(" in A dtor \n"); + } + } ObjA; + if (i == 0) + *NullPtr = 0; + + struct B { + ~B() { + printf(" in B dtor \n"); + } + } ObjB; + if (i == 1) + *NullPtr = 0; + + struct C { + ~C() { + printf(" in C dtor \n"); + } + } ObjC; + if (i == 2) + *NullPtr = 0; +} + +#define TRY __try +#define CATCH_ALL __except (1) + +int g; +int main() { + for (int i = 0; i < 3; i++) { + TRY { + crash(i); + } + CATCH_ALL { + printf(" Test CPP unwind: in catch handler i = %d \n", i); + } + } + return 0; +} diff --git a/clang/test/CodeGen/windows-seh-EHa-TryInFinally.cpp b/clang/test/CodeGen/windows-seh-EHa-TryInFinally.cpp new file mode 100644 --- /dev/null +++ b/clang/test/CodeGen/windows-seh-EHa-TryInFinally.cpp @@ -0,0 +1,42 @@ +// RUN: %clang_cc1 -triple x86_64-windows fasync-exceptions -fcxx-exceptions -fexceptions -fms-extensions -x c++ -Wno-implicit-function-declaration -S -emit-llvm %s -o - | FileCheck %s + +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: %[[src:[0-9-]+]] = load volatile i32, i32* %i +// CHECK-NEXT: i32 %[[src]] +// CHECK: invoke void @llvm.seh.try.end() +// CHECK: invoke void @llvm.seh.try.end() + +// CHECK: define internal void @"?fin$0@0@main@@"(i8 %abnormal_termination +// CHECK: invoke void @llvm.seh.try.begin() +// CHECK: invoke void @llvm.seh.try.end() + +// ***************************************************************************** +// Abstract: Test __Try in __finally under SEH -EHa option +void printf(...); +int volatile *NullPtr = 0; +int main() { + for (int i = 0; i < 3; i++) { + printf(" --- Test _Try in _finally --- i = %d \n", i); + __try { + __try { + printf(" In outer _try i = %d \n", i); + if (i == 0) + *NullPtr = 0; + } __finally { + __try { + printf(" In outer _finally i = %d \n", i); + if (i == 1) + *NullPtr = 0; + } __finally { + printf(" In Inner _finally i = %d \n", i); + if (i == 2) + *NullPtr = 0; + } + } + } __except (1) { + printf(" --- In outer except handler i = %d \n", i); + } + } + return 0; +} diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst --- a/llvm/docs/LangRef.rst +++ b/llvm/docs/LangRef.rst @@ -11530,6 +11530,68 @@ the escaped allocas are allocated, which would break attempts to use '``llvm.localrecover``'. +'``llvm.seh.try.begin``' and '``llvm.seh.try.end``' Intrinsics +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Syntax: +""""""" + +:: + + declare void @llvm.seh.try.begin() + declare void @llvm.seh.try.end() + +Overview: +""""""""" + +The '``llvm.seh.try.begin``' and '``llvm.seh.try.end``' intrinsics mark +the boundary of a _try region for Windows SEH Asynchrous Exception Handling. + +Semantics: +"""""""""" + +When a C-function is compiled with Windows SEH Asynchrous Exception option, +-feh_asynch (aka MSVC -EHa), these two intrinsics are injected to mark _try +boundary and to prevent from potential exceptions being moved across boundary. +Any set of operations can then be confined to the region by reading their leaf +inputs via volatile loads and writing their root outputs via volatile stores. + +'``llvm.seh.scope.begin``' and '``llvm.seh.scope.end``' Intrinsics +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Syntax: +""""""" + +:: + + declare void @llvm.seh.scope.begin() + declare void @llvm.seh.scope.end() + +Overview: +""""""""" + +The '``llvm.seh.scope.begin``' and '``llvm.seh.scope.end``' intrinsics mark +the boundary of a CPP object lifetime for Windows SEH Asynchrous Exception +Handling (MSVC option -EHa). + +Semantics: +"""""""""" + +LLVM's ordinary exception-handling representation associates EH cleanups and +handlers only with ``invoke``s, which normally correspond only to call sites. To +support arbitrary faulting instructions, it must be possible to recover the current +EH scope for any instruction. Turning every operation in LLVM that could fault +into an ``invoke`` of a new, potentially-throwing intrinsic would require adding a +large number of intrinsics, impede optimization of those operations, and make +compilation slower by introducing many extra basic blocks. These intrinsics can +be used instead to mark the region protected by a cleanup, such as for a local +C++ object with a non-trivial destructor. ``llvm.seh.scope.begin`` is used to mark +the start of the region; it is always called with ``invoke``, with the unwind block +being the desired unwind destination for any potentially-throwing instructions +within the region. `llvm.seh.scope.end` is used to mark when the scope ends +and the EH cleanup is no longer required (e.g. because the destructor is being +called). + .. _int_read_register: .. _int_write_register: diff --git a/llvm/include/llvm/IR/Intrinsics.td b/llvm/include/llvm/IR/Intrinsics.td --- a/llvm/include/llvm/IR/Intrinsics.td +++ b/llvm/include/llvm/IR/Intrinsics.td @@ -456,6 +456,16 @@ [llvm_ptr_ty, llvm_ptr_ty], [IntrNoMem]>; +// To mark the beginning/end of a try-scope for Windows SEH -EHa +// calls/invokes to these intrinsics are placed to model control flows +// caused by HW exceptions under option -EHa. +// calls/invokes to these intrinsics will be discarded during a codegen pass +// after EH tables are generated +def int_seh_try_begin : Intrinsic<[], [], [IntrReadMem, IntrWriteMem, IntrWillReturn]>; +def int_seh_try_end : Intrinsic<[], [], [IntrReadMem, IntrWriteMem, IntrWillReturn]>; +def int_seh_scope_begin : Intrinsic<[], [], [IntrNoMem]>; +def int_seh_scope_end : Intrinsic<[], [], [IntrNoMem]>; + // Note: we treat stacksave/stackrestore as writemem because we don't otherwise // model their dependencies on allocas. def int_stacksave : Intrinsic<[llvm_ptr_ty]>, diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -2789,6 +2789,10 @@ llvm_unreachable("Cannot invoke this intrinsic"); case Intrinsic::donothing: // Ignore invokes to @llvm.donothing: jump directly to the next BB. + case Intrinsic::seh_try_begin: + case Intrinsic::seh_scope_begin: + case Intrinsic::seh_try_end: + case Intrinsic::seh_scope_end: break; case Intrinsic::experimental_patchpoint_void: case Intrinsic::experimental_patchpoint_i64: @@ -6583,6 +6587,10 @@ lowerCallToExternalSymbol(I, FunctionName); return; case Intrinsic::donothing: + case Intrinsic::seh_try_begin: + case Intrinsic::seh_scope_begin: + case Intrinsic::seh_try_end: + case Intrinsic::seh_scope_end: // ignore return; case Intrinsic::experimental_stackmap: diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp --- a/llvm/lib/IR/Verifier.cpp +++ b/llvm/lib/IR/Verifier.cpp @@ -4223,6 +4223,10 @@ Assert( !F->isIntrinsic() || isa(I) || F->getIntrinsicID() == Intrinsic::donothing || + F->getIntrinsicID() == Intrinsic::seh_try_begin || + F->getIntrinsicID() == Intrinsic::seh_try_end || + F->getIntrinsicID() == Intrinsic::seh_scope_begin || + F->getIntrinsicID() == Intrinsic::seh_scope_end || F->getIntrinsicID() == Intrinsic::coro_resume || F->getIntrinsicID() == Intrinsic::coro_destroy || F->getIntrinsicID() == Intrinsic::experimental_patchpoint_void ||