Sirraide wrote:
CI failure looks unrelated.
https://github.com/llvm/llvm-project/pull/97910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/smanna12 closed
https://github.com/llvm/llvm-project/pull/97910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
smanna12 wrote:
Thank you for the reviews @Sirraide
https://github.com/llvm/llvm-project/pull/97910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide approved this pull request.
LGTM
Looks like we’re doing this null check pretty much in every other place
`SubstType` is used, so this makes sense.
https://github.com/llvm/llvm-project/pull/97910
___
cfe-commits mailing lis
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (smanna12)
Changes
This patch addresses static analyzer concern where TSI could be dereferenced
after being assigned a null value from SubstType in
clang::TemplateDeclInstantiator::VisitUsingEnumDecl(clang::UsingEnumDecl *).
The fix
https://github.com/smanna12 created
https://github.com/llvm/llvm-project/pull/97910
This patch addresses static analyzer concern where TSI could be dereferenced
after being assigned a null value from SubstType in
clang::TemplateDeclInstantiator::VisitUsingEnumDecl(clang::UsingEnumDecl *).
The
