[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