[llvm-commits] [llvm] r81082 - /llvm/trunk/tools/opt/opt.cpp

[Daniel Dunbar]

Looking at llvm-as -d, it is a slightly different feature, it dumps
the asm in addition to writing the output. Since this is a different
use case, I now think its better to leave it alone.

 - Daniel

On Sat, Sep 5, 2009 at 6:06 PM, Chris Lattner<clattner at apple.com> wrote:
>
> On Sep 5, 2009, at 4:58 PM, Daniel Dunbar wrote:
>
>> On Sat, Sep 5, 2009 at 10:16 AM, Nick Lewycky<nicholas at mxc.ca> wrote:
>>>
>>> Daniel Dunbar wrote:
>>>>
>>>> Author: ddunbar
>>>> Date: Sat Sep  5 06:34:53 2009
>>>> New Revision: 81082
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=81082&view=rev
>>>> Log:
>>>> opt: Add -S option to print output as LLVM assembly.
>>>
>>> llvm-as calls this flag "-d". Pick either, but please be consistent.
>>
>> Thanks Nick,
>>
>> I didn't realize this already existed in a tool. I think -S is a
>> significantly better choice than -d, but it might not be the best
>> choice since it can be confused with the driver -S option. Anyone have
>> a better suggestion? I'll wait for some opinions then change the tools
>> to be consistent.
>
> Feel free to rename llvm-as -d, it is just there to test the parser when the
> bitcode reader/writer support isn't up.
>
> -Chris
>