[llvm-commits] [llvm] r90113 - /llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp

Nick Lewycky nicholas at mxc.ca
Mon Nov 30 18:41:51 PST 2009


Chris Lattner wrote:
>
> On Nov 29, 2009, at 11:05 PM, Nick Lewycky wrote:
>
>> Author: nicholas
>> Date: Mon Nov 30 01:05:51 2009
>> New Revision: 90113
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=90113&view=rev
>> Log:
>> Revert r90107, fixing test/Transforms/GVN/2009-11-29-ReverseMap.ll and
>> the
>> llvm-gcc build.
>
> Uh, no it's not. You're thinking of the previous patch?

Yes it is. It's a revert of r90107: 
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20091123/091931.html

Nick

>
> -Chris
>
>>
>> Modified:
>> llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
>>
>> Modified: llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp?rev=90113&r1=90112&r2=90113&view=diff
>>
>>
>> ==============================================================================
>>
>> --- llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp (original)
>> +++ llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp Mon Nov 30
>> 01:05:51 2009
>> @@ -1156,21 +1156,8 @@
>> // that predecessor. We can still do PRE of the load, which would insert
>> // a computation of the pointer in this predecessor.
>> if (PredPtr == 0) {
>> - // Add the entry to the Result list.
>> - NonLocalDepEntry Entry(Pred,
>> - MemDepResult::getClobber(Pred->getTerminator()));
>> - Result.push_back(Entry);
>> -
>> - // Add it to the cache for this CacheKey so that subsequent queries get
>> - // this result.
>> - Cache = &NonLocalPointerDeps[CacheKey].second;
>> - MemoryDependenceAnalysis::NonLocalDepInfo::iterator It =
>> - std::upper_bound(Cache->begin(), Cache->end(), Entry);
>> - Cache->insert(It, Entry);
>> - Cache = 0;
>> -
>> - // Add it to the reverse map next.
>> - ReverseNonLocalPtrDeps[Pred->getTerminator()].insert(CacheKey);
>> + Result.push_back(NonLocalDepEntry(Pred,
>> + MemDepResult::getClobber(Pred->getTerminator())));
>> continue;
>> }
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>




More information about the llvm-commits mailing list