[llvm-commits] CVS: llvm/test/Transforms/LoopRotate/PhiRename-1.ll

[Chris Lattner]

On Apr 9, 2007, at 4:47 PM, Reid Spencer wrote:

> On Mon, 2007-04-09 at 18:40 -0500, Devang Patel wrote:
>>
>> Changes in directory llvm/test/Transforms/LoopRotate:
>>
>> PhiRename-1.ll updated: 1.1 -> 1.2
>> ---
>> Log message:
>>
>> Add check for opt crash.
>
> This idiom is becoming more and more prevalent. While it works, its  
> also
> doubling the runtime of each test its used in. I'm wondering if we  
> could
> invest in a "notgrep" script (since we're invoking "not" anyway) that
> would fail on empty input. Is this sufficient to avoid having to do
> things twice?

I'm not worried about it.  If you're worried about testing time, I'd  
suggest changing tests like CodeGen/ARM/vfp.ll to use %t to only  
invoke llc once :)

-Chris