[llvm-commits] [llvm] r166770 - /llvm/trunk/lib/Target/PowerPC/PPCInstr64Bit.td

[Will Schmidt]

On Fri, 2012-10-26 at 16:44 -0400, Rafael Espíndola wrote:
> > For reference, on the asmparser front, I only got far enough to get hung
> > up on PointerLikeRegClass support within tablegen.   (i.e.
> > http://llvm.org/bugs/show_bug.cgi?id=14011 "tblgen support for ptr_rc /
> > PointerLikeRegClass incomplete for gen-asm-matcher " ).   Attempts to
> > work around that by wrapping the ptr_rc references within something
> > Operand derived led to other errors, so I wasn't sure that was the right
> > approach.
> > ( http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-September/053566.html
> > )
> 
> I am sorry I am not familiar with tablegen and unable to help you in
> here, but we really have to figure something out. Having a backend
> with direct object emission and no testing is a recipe for disaster.

> If you want, check the early days of X86 ELF support where every fix I
> wrote caused some other bug which Roman found. Things only got better
> with a lot of asm -> .o testing in make check.

I don't doubt it.   I've been wishing this was working for a while now.

> How much do do you need to test this? Can you enable just a few
> instructions in the parser to avoid the tablegen bug?

The ppc64 asmparser subdirectory is currently non-existent.  I first hit
this when beginning to populate that code, but that was not actually
necessary to hit the problem.   This error occurs with tablegen
--gen-asm-matcher run against the existing .td files.









> 
> > Thanks,
> > -Will
> 
> Cheers,
> Rafael
>