[cfe-commits] r59932 - /cfe/trunk/lib/Analysis/RegionStore.cpp

Zhongxing Xu xuzhongxing at gmail.com
Sun Nov 23 18:30:48 PST 2008


Author: zhongxingxu
Date: Sun Nov 23 20:30:48 2008
New Revision: 59932

URL: http://llvm.org/viewvc/llvm-project?rev=59932&view=rev
Log:
Add a comment about the signedness.

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

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

==============================================================================
--- cfe/trunk/lib/Analysis/RegionStore.cpp (original)
+++ cfe/trunk/lib/Analysis/RegionStore.cpp Sun Nov 23 20:30:48 2008
@@ -273,6 +273,8 @@
 
   if (const StringRegion* SR = dyn_cast<StringRegion>(R)) {
     const StringLiteral* Str = SR->getStringLiteral();
+    // We intentionally made the size value signed because it participates in 
+    // operations with signed indices.
     return NonLoc::MakeVal(getBasicVals(), Str->getByteLength(), false);
   }
 





More information about the cfe-commits mailing list