[clang] [RISCV] Reduce the size of the index used for RVV intrinsics. NFC (PR #74906)
Craig Topper via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 12 10:16:49 PST 2023
topperc wrote:
> LGTM - though maybe use uint32_t?
>
> Looking at this code, the whole Intrinsics map vs OverloadIntrinsic map structure loops to have heavy redundancy and could be greatly simplified. Maybe a follow up?
Did you have a specific idea in mind? I'm skeptical that the `8` is the correct size for the SmallVector in OverloadedIntrinsicMap. Maybe we could use a single map and use the size of the vector being more than 1 to detect overloaded?
https://github.com/llvm/llvm-project/pull/74906
More information about the cfe-commits
mailing list