search for: commentdelimit

...100644 --- a/utils/TableGen/AsmMatcherEmitter.cpp +++ b/utils/TableGen/AsmMatcherEmitter.cpp @@ -1526,7 +1526,7 @@ void AsmMatcherInfo::buildInfo() { II->initialize(*this, SingletonRegisters, Variant, HasMnemonicFirst); // Validate the alias definitions. - II->validate(CommentDelimiter, false); + II->validate(CommentDelimiter, true); Matchables.push_back(std::move(II)); } On 2017-12-15 03:40, Daniel Sanders wrote: > Hi, > > On Instructions you can use checkEarlyTargetMatchPredicate() to check > that the operands are the same. There's a...

Hi, On Instructions you can use checkEarlyTargetMatchPredicate() to check that the operands are the same. There's an example of that in MipsAsmParser.cpp for DATI and DAHI. I can't think of a reason TableGen couldn't be made to allow this for InstAlias too. > On 15 Dec 2017, at 02:12, via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > Hello, > > InstAlias

Hello, InstAlias does not allow tied operands (repeated operands) in the asm string to be matched. It seems this situation is explicitly prevented in AsmMatcherEmitter.cpp: if (!Hack) PrintFatalError(TheDef->getLoc(), "ERROR: matchable with tied operand '" + Tok + "' can never be matched!");

...===// > +def AMDILAsmWriter : AsmWriter { > + string AsmWriterClassName = "AsmPrinter"; > + int Variant = 0; > +} > + > +def AMDILAsmParser : AsmParser { > + string AsmParserClassName = "AsmParser"; > + int Variant = 0; > + > + string CommentDelimiter = ";"; > + > + string RegisterPrefix = "r"; > + > +} > + > + > +def AMDIL : Target { > + // Pull in Instruction Info: > + let InstructionSet = AMDILInstrInfo; > + let AssemblyWriters = [AMDILAsmWriter]; > + let AssemblyParsers = [AMDILAs...