From 0b4c2ee3810c047afd415c9fd328798854ef96ae Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Sun, 3 Nov 2019 11:33:57 +0000 Subject: ModuleMap::findHeader - fix null dereference warning. NFCI. We were checking M for a null value after we'd already dereferenced it multiple times. --- clang/lib/Lex/ModuleMap.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'clang/lib/Lex/ModuleMap.cpp') diff --git a/clang/lib/Lex/ModuleMap.cpp b/clang/lib/Lex/ModuleMap.cpp index db59629..fe20a35 100644 --- a/clang/lib/Lex/ModuleMap.cpp +++ b/clang/lib/Lex/ModuleMap.cpp @@ -229,7 +229,7 @@ const FileEntry *ModuleMap::findHeader( llvm::sys::path::append(FullPathName, RelativePathName); auto *NormalHdrFile = GetFile(FullPathName); - if (M && !NormalHdrFile && Directory->getName().endswith(".framework")) { + if (!NormalHdrFile && Directory->getName().endswith(".framework")) { // The lack of 'framework' keyword in a module declaration it's a simple // mistake we can diagnose when the header exists within the proper // framework style path. -- cgit v1.1