[polly] r302696 - [Fix] [Fortran Support] Fix variable name & make testcase activate on release

Siddharth Bhat via llvm-commits llvm-commits at lists.llvm.org
Wed May 10 10:27:48 PDT 2017


Author: bollu
Date: Wed May 10 12:27:48 2017
New Revision: 302696

URL: http://llvm.org/viewvc/llvm-project?rev=302696&view=rev
Log:
[Fix] [Fortran Support] Fix variable name & make testcase activate on release

There was:
    #ifdef NDEBUG

This should be:
    #ifndef NDEBUG

Also, the variable name was incorrect. Fixed the variable name.

Modified:
    polly/trunk/lib/Analysis/ScopInfo.cpp

Modified: polly/trunk/lib/Analysis/ScopInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopInfo.cpp?rev=302696&r1=302695&r2=302696&view=diff
==============================================================================
--- polly/trunk/lib/Analysis/ScopInfo.cpp (original)
+++ polly/trunk/lib/Analysis/ScopInfo.cpp Wed May 10 12:27:48 2017
@@ -1039,8 +1039,8 @@ void MemoryAccess::setFortranArrayDescri
 
 // TODO: write checks to make sure it looks _exactly_ like a Fortran array
 // descriptor
-#ifdef NDEBUG
-  StructType *ty = dyn_cast<StructType>(Descriptor->getValueType());
+#ifndef NDEBUG
+  StructType *ty = dyn_cast<StructType>(FAD->getValueType());
   assert(ty && "expected value of type Fortran array descriptor");
   assert(ty->hasName() && ty->getName().startswith("struct.array") &&
          "expected global to follow Fortran array descriptor type naming "




More information about the llvm-commits mailing list