[PATCH] Use the integrated assembler by default on OpenBSD/powerpc

Brad Smith brad at comstyle.com
Fri Mar 7 01:49:49 PST 2014


On 04/03/14 9:08 PM, Brad Smith wrote:
> On Tue, Mar 04, 2014 at 01:33:48PM +0000, Daniel Sanders wrote:
>> You'll find four  'UseIntegratedAssembler = true;' statements in
>> lib/Target/X86/MCTargetDesc/X86MCAsmInfo.cpp, one in each
>> MCAsmInfo subclass. These enable the integrated assembler by
>> default for X86. There are similar statements in other subclasses
>> of MCAsmInfo.
>>
>> The PowerPC version of this code (in PPCMCAsmInfo.cpp) currently
>> looks like this:
>> 	  if (T.getOS() == llvm::Triple::FreeBSD ||
>> 	      (T.getOS() == llvm::Triple::NetBSD && !is64Bit))
>> 	    UseIntegratedAssembler = true;
>
> Thank you for pointing that out. I have submitted a patch to the
> LLVM list.

The LLVM patch was commited. Please commit this patch.


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.




More information about the cfe-commits mailing list