From ddfbdbfefae04ea71391a38ed5e9cb6975f6630b Mon Sep 17 00:00:00 2001 From: Adam Czachorowski Date: Thu, 15 Apr 2021 22:33:05 +0200 Subject: [PATCH] [clang] Do not crash on template specialization following a fatal error There was a missing isInvalid() check leading to an attempt to instantiate template with an empty instantiation stack. Differential Revision: https://reviews.llvm.org/D100675 --- clang/lib/Sema/SemaTemplateDeduction.cpp | 3 +++ clang/test/SemaCXX/template-specialization-fatal.cpp | 11 +++++++++++ 2 files changed, 14 insertions(+) create mode 100644 clang/test/SemaCXX/template-specialization-fatal.cpp diff --git a/clang/lib/Sema/SemaTemplateDeduction.cpp b/clang/lib/Sema/SemaTemplateDeduction.cpp index d755696..7d548e4 100644 --- a/clang/lib/Sema/SemaTemplateDeduction.cpp +++ b/clang/lib/Sema/SemaTemplateDeduction.cpp @@ -5466,6 +5466,9 @@ static bool isAtLeastAsSpecializedAs(Sema &S, QualType T1, QualType T2, Deduced.end()); Sema::InstantiatingTemplate Inst(S, Info.getLocation(), P2, DeducedArgs, Info); + if (Inst.isInvalid()) + return false; + auto *TST1 = T1->castAs(); bool AtLeastAsSpecialized; S.runWithSufficientStackSpace(Info.getLocation(), [&] { diff --git a/clang/test/SemaCXX/template-specialization-fatal.cpp b/clang/test/SemaCXX/template-specialization-fatal.cpp new file mode 100644 index 0000000..2191e54 --- /dev/null +++ b/clang/test/SemaCXX/template-specialization-fatal.cpp @@ -0,0 +1,11 @@ +// RUN: %clang_cc1 -verify -fsyntax-only %s +// Verify clang doesn't assert()-fail on template specialization happening after +// fatal error. + +#include "not_found.h" // expected-error {{'not_found.h' file not found}} + +template +struct foo {}; + +template +struct foo(0)(static_cast(0)()))> {}; -- 2.7.4