[all-commits] [llvm/llvm-project] 4b25c1: [SLP]Fix an assertion for the size of user nodes.

cilkplus via All-commits all-commits at lists.llvm.org
Fri Jul 30 05:47:23 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4b25c113210e579a5346ca0abc0717ab1ce5d9df
      https://github.com/llvm/llvm-project/commit/4b25c113210e579a5346ca0abc0717ab1ce5d9df
  Author: Alexey Bataev <a.bataev at outlook.com>
  Date:   2021-07-30 (Fri, 30 Jul 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
    A llvm/test/Transforms/SLPVectorizer/AArch64/reorder-fmuladd-crash.ll

  Log Message:
  -----------
  [SLP]Fix an assertion for the size of user nodes.

For the nodes with reused scalars the user may be not only of the size
of the final shuffle but also of the size of the scalars themselves,
need to check for this. It is safe to just modify the check here, since
the order of the scalars themselves is preserved, only indeces of the
reused scalars are changed. So, the users with the same size as the
number of scalars in the node, will not be affected, they still will get
the operands in the required order.

Reported by @mstorsjo in D105020.

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




More information about the All-commits mailing list