[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