[all-commits] [llvm/llvm-project] 2f8f01: [SLP] Add test case showing shortcoming in honorin...

[Björn Pettersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2f8f01dcb3d43d2fb1149fc8988e61f93f9064f5
      https://github.com/llvm/llvm-project/commit/2f8f01dcb3d43d2fb1149fc8988e61f93f9064f5
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2021-03-22 (Mon, 22 Mar 2021)

  Changed paths:
    A llvm/test/Transforms/SLPVectorizer/slp-max-reg-size.ll

  Log Message:
  -----------
  [SLP] Add test case showing shortcoming in honoring max reg size


  Commit: 688cdddafb0dfdeb5f3c5e1e22b88a0cdfc54c0c
      https://github.com/llvm/llvm-project/commit/688cdddafb0dfdeb5f3c5e1e22b88a0cdfc54c0c
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2021-03-22 (Mon, 22 Mar 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
    M llvm/test/Transforms/SLPVectorizer/slp-max-reg-size.ll

  Log Message:
  -----------
  [SLP] Honor min/max regsize and min/max VF in vectorizeStores

Make sure we use PowerOf2Floor instead of PowerOf2Ceil when
calculating max number of elements that fits inside a vector
register (otherwise we could end up creating vectors larger
than the maximum vector register size).

Also make sure we honor the min/max VF (as given by TTI or
cmd line parameters) when doing vectorizeStores.

Reviewed By: anton-afanasyev

Differential Revision: https://reviews.llvm.org/D97691


Compare: https://github.com/llvm/llvm-project/compare/9c16621c0d4f...688cdddafb0d