nickdesaulniers created this revision.
nickdesaulniers added reviewers: rsmith, sfantao.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

This was flagged in https://www.viva64.com/en/b/0629/ under "Snippet No.
47".

Looks like TC is checked for nullptr once, but not the second time.
Fixes r275645.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D62232

Files:
  clang/lib/Driver/Driver.cpp


Index: clang/lib/Driver/Driver.cpp
===================================================================
--- clang/lib/Driver/Driver.cpp
+++ clang/lib/Driver/Driver.cpp
@@ -1821,7 +1821,8 @@
           else
             os << "host";
           os << " (";
-          os << TC->getTriple().normalize();
+          if (TC)
+            os << TC->getTriple().normalize();
 
           if (BoundArch)
             os << ":" << BoundArch;


Index: clang/lib/Driver/Driver.cpp
===================================================================
--- clang/lib/Driver/Driver.cpp
+++ clang/lib/Driver/Driver.cpp
@@ -1821,7 +1821,8 @@
           else
             os << "host";
           os << " (";
-          os << TC->getTriple().normalize();
+          if (TC)
+            os << TC->getTriple().normalize();
 
           if (BoundArch)
             os << ":" << BoundArch;
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D62232: [Clang][D... Nick Desaulniers via Phabricator via cfe-commits

Reply via email to