[LLVMdev] (MC) Register parsing for AsmParser (standalone assembler)

[Chris Lattner]

On Jan 31, 2012, at 1:26 PM, Carter, Jack wrote:

> I'm trying to build a standalone assembler for Mips using AsmParser.
> 
> Following the lead of X86, ARM and MBlaze I have run tblgen -gen-asm-matcher on Mips.td to produce tables and methods to aid the parser (MipsAsmParser.cpp) which is a stripped down ARM implementation.
> 
> I am getting an assertion for what I believe are multiple register definitions with the same name.
> 
> llvm-tblgen: /home/jcarter/workarea/asm/llvm/utils/TableGen/StringMatcher.cpp:52: bool llvm::StringMatcher::EmitStringMatcherForChar(const std::vector<const std::pair<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::basic_string<char, std::char_traits<char>, std::allocator<char> > >*, std::allocator<const std::pair<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::basic_string<char, std::char_traits<char>, std::allocator<char> > >*> >&, unsigned int, unsigned int) const: Assertion `Matches.size() == 1 && "Had duplicate keys to match on"' failed.

I don't know the actual answer for your question, but it would be really great if you could add code to tblgen to have it produce an error for this case, instead of aborting.

-Chris

> 
> *********************************
> Background for Mips register sets:
> *********************************
> 
> The Mips architecture has register names that are context sensitive. 
> 
> For instance, the 32 general purpose registers for both Mips32 and Mips64 have the same name, but each of the Mips32 registers are just a subregister in their Mips64 instance.
> 
>   def AT   : MipsGPRReg< 1, "AT">,   DwarfRegNum<[1]>;
>   def AT_64   : Mips64GPRReg< 1, "AT",   [AT]>;
> 
> It gets more interesting with floating point where we have 3 different configurations, single precision, double precision aliased with single precision pair and straight double point precision. All of which share the same register name.
> 
>   /// Mips Single point precision FPU Registers
>   def F0  : FPR< 0,  "F0">, DwarfRegNum<[32]>;
> 
>   /// Mips Double point precision FPU Registers (aliased
>   /// with the single precision to hold 64 bit values)
>   def D0  : AFPR< 0,  "F0", [F0,   F1]>;
> 
>   /// Mips Double point precision FPU Registers in MFP64 mode.
>   def D0_64  : AFPR64<0, "F0", [F0]>;
> 
> Notice that we currently need the symbolic name to be different (F0/D0/D0_64) for use in the codegen.
> 
> The examples here are from lib/Target/Mips/MipsRegisterInfo.td.
> 
> Do I just need to use/write another register parser? Or is there a clever way of defining the context sensitive Mips register set?
> 
> Thanks,
> 
> Jack
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120201/204b4fee/attachment.html>