diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-11-03 11:33:57 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-11-03 11:33:57 +0000 |
| commit | 0b4c2ee3810c047afd415c9fd328798854ef96ae (patch) | |
| tree | edebb4b4608c424c45975dc0dfdfddfeafd9d7d8 /clang/lib/Lex/ModuleMap.cpp | |
| parent | 3823759afbc10b5c08b1f7c0f1086c63a1ffca32 (diff) | |
| download | bcm5719-llvm-0b4c2ee3810c047afd415c9fd328798854ef96ae.tar.gz bcm5719-llvm-0b4c2ee3810c047afd415c9fd328798854ef96ae.zip | |
ModuleMap::findHeader - fix null dereference warning. NFCI.
We were checking M for a null value after we'd already dereferenced it multiple times.
Diffstat (limited to 'clang/lib/Lex/ModuleMap.cpp')
| -rw-r--r-- | clang/lib/Lex/ModuleMap.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Lex/ModuleMap.cpp b/clang/lib/Lex/ModuleMap.cpp index db59629997e..fe20a350703 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. |
