[LLVMdev] [PATCH] Support asm comment output

Chris Lattner clattner at apple.com
Mon Jul 13 11:05:04 PDT 2009


On Jul 13, 2009, at 10:07 AM, David Greene wrote:

> On Sunday 12 July 2009 18:56, Chris Lattner wrote:
>
>> A couple of things are important to discuss:
>>
>> +    dynamic_cast<formatted_raw_ostream &>(O) << Comment(*c);
>>
>> We're trying to eliminate rtti, please don't add new uses of it.
>> Switching all of the asmprinter to statically use
>> formatted_raw_ostream would be appropriate.
>
> To separate comments, would it be ok to commit this bit as-is and  
> follow
> it up with making the change to static formatted_raw_ostream?

Yep, I'm fine with it being that way temporarily, please don't add the  
comments vector to MachineInstr though.

-Chris



More information about the llvm-dev mailing list