[llvm-commits] [llvm] r122343 - /llvm/trunk/lib/Transforms/Utils/SimplifyInstructions.cpp

[Chris Lattner]

On Dec 21, 2010, at 12:18 PM, Duncan Sands wrote:

> Hi Chris,
> 
>>> Visit instructions deterministically.  Use a FIFO so as to approximately
>>> visit instructions before their uses, since InstructionSimplify does a
>>> better job in that case.  All this prompted by Frits van Bommel.
>> 
>> Ok.  If you're going to delay deleting instructions, it might be worthwhile to drop their operands so that "dead" instructions don't cause hasOneUse() checks to fail.
> 
> since InstructionSimplify doesn't pay any attention to uses (it doesn't need
> to since it only returns existing instructions) this doesn't matter.

Ok!