[LLVMdev] making emitInlineAsm protected
echristo at gmail.com
Fri Jan 31 09:19:56 PST 2014
Mostly raw text support should only be used for things like comments
that don't affect the overall output in the object file either way. I
know that Rafael and I want to remove any other use.
On Fri, Jan 31, 2014 at 2:26 AM, Daniel Sanders
<Daniel.Sanders at imgtec.com> wrote:
> It may be moot because Reed is currently rewriting the patch to avoid using EmitInlineAsm and EmitRawText but I wanted to question something here.
> I'm thinking that hasRawTextSupport() shouldn't be the condition used inside EmitInlineAsm. I think it would be more correct to have a useRawTextSupport() predicate that can return hasRawTextSupport() for (sub)targets that haven't implemented their MC layer, and false for those that have. What do you think?
>> -----Original Message-----
>> From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu]
>> On Behalf Of Rafael Espíndola
>> Sent: 29 January 2014 20:14
>> To: Reed Kotler
>> Cc: LLVMdev at cs.uiuc.edu
>> Subject: Re: [LLVMdev] making emitInlineAsm protected
>> On 28 January 2014 19:56, reed kotler <rkotler at mips.com> wrote:
>> > I would like to make the following member of AsmPrinter be protected
>> > void EmitInlineAsm(StringRef Str, const MDNode *LocMDNode = 0,
>> > InlineAsm::AsmDialect AsmDialect =
>> > InlineAsm::AD_ATT) const;
>> > I have some stubs that I want to emit in MipsAsmParser .
>> You mean Printer? There is no such a thing as inline asm in a .s file.
>> > Are there any objections to doing this?
>> Probably. EmitInilneAsm is the only case I know that has a reasonable use of
>> hasRawTextSupport in order for it to work on targets without an asm parser.
>> Exposing it exposes a way for targets avoid the MC layer.
>> LLVM Developers mailing list
>> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
More information about the llvm-dev