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

[Duncan Sands]

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.

Ciao,

Duncan.