r278733 - [CUDA] Fix CUDA install version parsing.
    Justin Lebar via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Aug 15 13:38:48 PDT 2016
    
    
  
Author: jlebar
Date: Mon Aug 15 15:38:48 2016
New Revision: 278733
URL: http://llvm.org/viewvc/llvm-project?rev=278733&view=rev
Log:
[CUDA] Fix CUDA install version parsing.
Summary:
getAsInteger returns true on error.  Oops.
No test because the behavior at the moment is identical with or without
this change.
Reviewers: tra
Subscribers: cfe-commits, Prazek
Differential Revision: https://reviews.llvm.org/D23340
Modified:
    cfe/trunk/lib/Driver/ToolChains.cpp
Modified: cfe/trunk/lib/Driver/ToolChains.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=278733&r1=278732&r2=278733&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains.cpp Mon Aug 15 15:38:48 2016
@@ -1730,8 +1730,8 @@ static CudaVersion ParseCudaVersionFile(
   int Major = -1, Minor = -1;
   auto First = V.split('.');
   auto Second = First.second.split('.');
-  if (!First.first.getAsInteger(10, Major) ||
-      !Second.first.getAsInteger(10, Minor))
+  if (First.first.getAsInteger(10, Major) ||
+      Second.first.getAsInteger(10, Minor))
     return CudaVersion::UNKNOWN;
 
   if (Major == 7 && Minor == 0) {
    
    
More information about the cfe-commits
mailing list