[cfe-commits] r62844 - /cfe/trunk/lib/Analysis/MemRegion.cpp

[Zhongxing Xu]

Author: zhongxingxu
Date: Fri Jan 23 04:19:29 2009
New Revision: 62844

URL: http://llvm.org/viewvc/llvm-project?rev=62844&view=rev
Log:
When getting the element region type, we should get the rvalue type of the super
region. Otherwise we would get an extra level of pointer type.

Modified:
    cfe/trunk/lib/Analysis/MemRegion.cpp

Modified: cfe/trunk/lib/Analysis/MemRegion.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Analysis/MemRegion.cpp?rev=62844&r1=62843&r2=62844&view=diff

==============================================================================
--- cfe/trunk/lib/Analysis/MemRegion.cpp (original)
+++ cfe/trunk/lib/Analysis/MemRegion.cpp Fri Jan 23 04:19:29 2009
@@ -108,8 +108,8 @@
 }
 
 QualType ElementRegion::getRValueType(ASTContext& C) const {
-  QualType T = getArrayRegion()->getLValueType(C);
-  // FIXME: Should ArrayType be considered an LValue or RValue type?
+  QualType T = getArrayRegion()->getRValueType(C);
+
   if (isa<ArrayType>(T.getTypePtr())) {
     ArrayType* AT = cast<ArrayType>(T.getTypePtr());
     return AT->getElementType();