[LLVMdev] making emitInlineAsm protected
Daniel.Sanders at imgtec.com
Fri Jan 31 02:26:01 PST 2014
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
More information about the llvm-dev