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

[bollu]

Summary

There was:

#ifdef NDEBUG

This should be:

#ifndef NDEBUG

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

[bollu]

@Meinersbur: I'm sorry for letting this slip past. I only build in debug mode, so naturally #ifdef NDEBUG will not be run. Is there either some simple way to also run release mode without rebuilding LLVM?

Also, is it possible to get an email from the buildbots in case a build fails? That would be nice, if possible.

[grosser]

buildbots should send an email.

This is a trivial patch, please just commit it directly.

[bollu]

fixed by 858d31018f9941d7e27ecfb2d0dbe5cefbce4efc

lib/Analysis/ScopInfo.cpp

raw_ostream &polly::operator<<(raw_ostream &OS,
   return OS;
 }
 
 void MemoryAccess::setFortranArrayDescriptor(GlobalValue *FAD) {
   this->FAD = FAD;
 
 // 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 "
          "convention");
   assert(ty->getNumElements() == 4 &&
          "expected layout to be like Fortran array descriptor type");
 #endif
 }