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 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
