[llvm] [LLVM][MC] Add support to cull inactive decoders in decoder emitter (PR #154865)
Rahul Joshi via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 26 12:46:50 PDT 2025
jurahul wrote:
> > ```
> > template <typename T>
> > constexpr bool isSupportedInsnBitWidth(uint32_t N) { return false; }
> > ```
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > This will allow using the same C++ type for > 1 bitwidths. Then we would need to remove the current specializations in MCDecoder.h as they assume a specific usage and then backends can choose. Not sure if that generalization is needed.
>
> LMK if this is something to pursue. I suspect it may not be too much effort to do this and offers additional flexibility for backends.
I did and now RISCV can continue using uint64_t for 48-bit insts. Will update the PR soon
https://github.com/llvm/llvm-project/pull/154865
More information about the llvm-commits
mailing list