Re: [PATCH 10/11] tcg/loongarch64: Import REVBIT insns
Philippe Mathieu-Daudé <[email protected]> Tue, 4 Aug 2026 04:51:26 +0200
| Newsgroups | org.nongnu.qemu-arm,org.nongnu.qemu-devel |
|---|---|
| Message-ID | <[email protected]> |
On 3/8/26 19:26, Richard Henderson wrote: > On 8/3/26 08:06, Philippe Mathieu-Daudé wrote: >>> * This file is auto-generated by genqemutcgdefs from >>> * https://github.com/loongson-community/loongarch-opcodes, >>> - * from commit 7f353fb69bd99ce6edfad7ad63948c4bb526f0bf. >>> + * from commit 40ea52087699392ad911387b666541891d33da31. >> >> I couldn't find it then noticed your >> https://github.com/loongson-community/loongarch-opcodes/pull/10, >> so pending it to be merged? > > Yes. > >>> -static int32_t __attribute__((unused)) >>> -encode_d_slot(LoongArchInsn opc, uint32_t d) >>> +static int32_t __attribute__((unused)) encode_d_slot(LoongArchInsn opc, >>> + uint32_t d) >> >> What triggered this code churn? Could we commit before generating for >> REVBIT opcodes? > > It looks like the program passes off to clang-format. Perhaps a version > change in the year since last regenerated? Then maybe call genqemutcgdefs to regenerate clang-formatted in a preliminary patch? Would also help reverts. > > > r~ >