[llvm-commits] [llvm] r67194 - in /llvm/trunk/lib/Transforms: Scalar/ScalarReplAggregates.cpp Utils/Local.cpp

Zhou Sheng zhousheng00 at gmail.com
Wed Mar 18 05:49:08 PDT 2009


2009/3/18 Duncan Sands <baldrick at free.fr>

> Hi,
>
> > The safety is checked by previous code of scalarrepl (function
> > isSafeUseOfAllocation),  the instruction "I" can be only load, store,
> > bitcast or getelementptr, so, here it is save to delete if it only used
> by
> > debug info.
>
> I see, thanks for the explanation.  In that case I think it would be better
> to explicitly check for the Store case.  With your change loads with no
> uses
> are no longer removed for example.


You are right. Done.


>
>
> Ciao,
>
> Duncan.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090318/36521a9b/attachment.html>


More information about the llvm-commits mailing list