[llvm-branch-commits] [llvm] [RISCV] Support memcmp expansion for vectors (PR #114517)

[Craig Topper via llvm-branch-commits]

================
@@ -2512,9 +2512,11 @@ bool RISCVTTIImpl::isProfitableToSinkOperands(
 RISCVTTIImpl::TTI::MemCmpExpansionOptions
 RISCVTTIImpl::enableMemCmpExpansion(bool OptSize, bool IsZeroCmp) const {
   TTI::MemCmpExpansionOptions Options;
+  // Here we assume that a core that has implemented unaligned vector access
+  // should also have implemented unaligned scalar access.
----------------
topperc wrote:

I'm not comfortable with this assumption. Is it even required? Aren't we doing byte element loads?

https://github.com/llvm/llvm-project/pull/114517