[llvm] [DSE] Optimizing shrinking of memory intrinsic (PR #106425)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 29 11:47:17 PDT 2024
goldsteinn wrote:
> > > * It looks like getLoadStoreVecRegBitWidth defaults to 128 and is not overridden on most targets. In the context of this patch, it will be misleading for any (sub-)targets without vectors. 128 bit is a good default assumption for a target that has vectors, but not for a scalar-only target.
> >
> >
> > Do you know how to test if a target supports vectors? I don't see any obvious knob.
>
> Might be worth to try
>
> ```
> /// \return The width of the largest scalar or vector register type. <<<
> TypeSize getRegisterBitWidth(RegisterKind K) const;
> ```
Did so, basically if scalar width >= vec width, assumed scalar width for width of loads/stores.
https://github.com/llvm/llvm-project/pull/106425
More information about the llvm-commits
mailing list