[PATCH] Demote EmitRawText call in AsmPrinter::EmitInlineAsm() and remove hasRawTextSupport() call
Rafael EspĂndola
rafael.espindola at gmail.com
Mon Feb 10 11:16:41 PST 2014
>> IMHO it is the MCAsmInfoDarwin that should be named MCAsmInfoMachO. The handling of COFF and MachO are probably correct enough to not need a FIXME :-)
> Great. Do you have any thoughts on the 'OS is Solaris' check?
>
> I think I can put it in SparcELFMCAsmInfo and X86ELFMCAsmInfo to cover most of it, but I'm not sure if there are other subclasses that need it.
That should do it. If people want in the future we can move it up to
MCAsmInfoELF by passing a triple down to the constructor.
Cheers,
Rafael
More information about the llvm-commits
mailing list