<div dir="ltr">Thanks for the review. I have attached an updated patch which uses "_MINUS_" instead of "_DASH_". I do not have commit rights so I need someone to commit it for me.<div><br></div><div>Thanks,</div><div><br></div><div>Jordy<br><br>On Wed, Jan 14, 2015 at 1:46 AM, Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>><br>> LGTM, although I'd prefer "_MINUS_" to "_DASH_" (since _MINUS_1 will likely be easier to interpret than _DASH_1).<br>><br>>  -Hal<br>><br>> ----- Original Message -----<br>> > From: "Jordy Potman" <<a href="mailto:jordypotman@gmail.com">jordypotman@gmail.com</a>><br>> > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>> > Sent: Sunday, January 11, 2015 11:37:56 AM<br>> > Subject: [PATCH] Add support for negative immediates to TableGen      AsmMatcherEmitter<br>> ><br>> > For the LLVM backend<br>> > (<a href="https://github.com/ucb-bar/riscv-llvm/tree/riscv-trunk">https://github.com/ucb-bar/riscv-llvm/tree/riscv-trunk</a>) for the<br>> > RISC-V architecture (<a href="http://riscv.org/">http://riscv.org/</a>) I would like to create an<br>> > InstAlias with a negative immediate, i.e.:<br>> ><br>> > def NOT : InstAlias<"not $dst, $src", (XORI GR32:$dst, GR32:$src,<br>> > -1)>;<br>> ><br>> > However the TableGen AsmMatcherEmitter currently generates invalid<br>> > enum values for negative immediates:<br>> ><br>> > lib/Target/RISCV/RISCVGenAsmMatcher.inc:95:11: error: expected '=<br>> > constant-expression' or end of enumerator definition<br>> >   CVT_imm_-1,<br>> >           ^<br>> ><br>> > The attached patch adds support for negative immediates to the<br>> > TableGen AsmMatcherEmitter.<br>> ><br>> > Jordy<br>> ><br>> > _______________________________________________<br>> > llvm-commits mailing list<br>> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>> ><br>><br>> --<br>> Hal Finkel<br>> Assistant Computational Scientist<br>> Leadership Computing Facility<br>> Argonne National Laboratory<br><br></div></div>