llvm dev - May 2008 - [LLVMdev] -loop-deletion commandline option inconsistent?

Hi,

the newly added LoopDeletion pass uses loop-deletion as it's name (and thus
commandline option for opt). Looking at the other loop passes (loop-rotate,
loop-unroll, loop-reduce), perhaps loop-delete would be a better name?

Gr.

Matthijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20080507/20f2e3c0/attachment.sig>

Hi,
> the newly added LoopDeletion pass uses loop-deletion as it's name (and
thus
> commandline option for opt). Looking at the other loop passes (loop-rotate,
> loop-unroll, loop-reduce), perhaps loop-delete would be a better name?
No opinions on this? If this is ever going to be changed, it's probably a
good
idea to do it before the 2.3 release, not after...

Gr.

Matthijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20080513/a42cb76a/attachment.sig>

>> the newly added LoopDeletion pass uses loop-deletion as it's name
(and thus
>> commandline option for opt). Looking at the other loop passes
(loop-rotate,
>> loop-unroll, loop-reduce), perhaps loop-delete would be a better name?
> No opinions on this? If this is ever going to be changed, it's probably
a good
> idea to do it before the 2.3 release, not after...
2.3 has already branched and loop delete is not on by default. This can be 
changed after.

-Tanya
>
> Gr.
>
> Matthijs
>