[llvm-branch-commits] [llvm] [RISCV] Add initial support of memcmp expansion (PR #107548)
Craig Topper via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Tue Nov 5 11:24:10 PST 2024
topperc wrote:
> > > Can we break the enabling down into more manageable pieces? I think `enableUnalignedScalarMem() && (Subtarget->hasStdExtZbb() || Subtarget->hasStdExtZbkb() || IsZeroCmp)` might be a good starting point.
> >
> >
> > I'd be fine with this type of approach.
>
> Thanks! I have applied this suggestion. For `ExpandMemcmp` issue, I will try to fix it later.
I meant we should start with only doing inline memcmp expansion under those conditions. Not just the overlapping loads part.
https://github.com/llvm/llvm-project/pull/107548
More information about the llvm-branch-commits
mailing list