[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