[llvm-commits] [Patch] XCoreBackend, part 4

Richard Osborne richard at xmos.com
Fri Nov 7 02:56:30 PST 2008


Anton Korobeynikov wrote:
> Hi, Richard
>
>   
>> This patch adds the TargetAsmInfo and AsmPrinter classes for the XCore
>> backend.
>>     
> These look ok for me, except that you don't need to use None in stuff
> like "SectionFlags::None | SectionFlags::Small", however it can be
> considered better for readability.
>   
Thanks for the feedback. I've made the changes you suggest and will commit.

-- 
Richard Osborne | XMOS
http://www.xmos.com





More information about the llvm-commits mailing list