[all-commits] [llvm/llvm-project] b58b38: [LV] NFC: Add new test

[Sander de Smalen via All-commits]

  Branch: refs/heads/users/sdesmalen-arm/lv-move-partial-reduction-condition
  Home:   https://github.com/llvm/llvm-project
  Commit: b58b38e190db23eef974314309cae4d14875339a
      https://github.com/llvm/llvm-project/commit/b58b38e190db23eef974314309cae4d14875339a
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/test/Transforms/LoopVectorize/AArch64/vplan-printing.ll

  Log Message:
  -----------
  [LV] NFC: Add new test

This shows that no VPExpression is built for partial reductions
that have some form of predication.


  Commit: c04d2191c5d0f05bfd5c0c1d3f9086677e0bdcae
      https://github.com/llvm/llvm-project/commit/c04d2191c5d0f05bfd5c0c1d3f9086677e0bdcae
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
    M llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
    M llvm/test/Transforms/LoopVectorize/AArch64/partial-reduce-dot-product-epilogue.ll
    M llvm/test/Transforms/LoopVectorize/AArch64/partial-reduce-dot-product-neon.ll
    M llvm/test/Transforms/LoopVectorize/AArch64/partial-reduce-dot-product.ll
    M llvm/test/Transforms/LoopVectorize/AArch64/vplan-printing.ll

  Log Message:
  -----------
  [LV] Move condition to VPPartialReductionRecipe::execute

This means that VPExpressions will now be constructed for
VPPartialReductionRecipe's when the loop has tail-folding predication.

Note that control-flow (if/else) predication is not yet handled
for partial reductions, because of the way partial reductions
are recognised and built up.


  Commit: 80e5448cddc0691af30fbffde7a050fb7b9fa3f3
      https://github.com/llvm/llvm-project/commit/80e5448cddc0691af30fbffde7a050fb7b9fa3f3
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

  Log Message:
  -----------
  Include the cost of the select/predication in the cost of the partial reduction.

In practice this won't make much difference, because VPExpressions already accounts
for the cost of the predication.


Compare: https://github.com/llvm/llvm-project/compare/46b0e9ce654d...80e5448cddc0

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications