[all-commits] [llvm/llvm-project] 44f216: [LV] Keep duplicate recipes in VPExpressionRecipe

[Sam Tebbs via All-commits]

  Branch: refs/heads/users/SamTebbs33/vpexpression-keep-duplicates
  Home:   https://github.com/llvm/llvm-project
  Commit: 44f216a28c0208c4c21be613cab10a62ad32a0b2
      https://github.com/llvm/llvm-project/commit/44f216a28c0208c4c21be613cab10a62ad32a0b2
  Author: Sam Tebbs <samuel.tebbs at arm.com>
  Date:   2025-09-19 (Fri, 19 Sep 2025)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/VPlan.h
    M llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
    M llvm/test/Transforms/LoopVectorize/vplan-printing-reductions.ll

  Log Message:
  -----------
  [LV] Keep duplicate recipes in VPExpressionRecipe

The VPExpressionRecipe class uses a set to store its bundled recipes. If
repeated recipes are bundled then the duplicates will be lost, causing
the following recipes to not be at the expected place in the set.

When printing a reduce.add(mul(ext, ext)) bundle, if the extends are the
same then the 3rd element of the set will the the reduction, rather than
the expected mul, causing a cast error. With this change, the recipes
are at the expected index in the set.

Fixes #156464


  Commit: 23005065ceb7551e80eeb4e2c24e6515234742e4
      https://github.com/llvm/llvm-project/commit/23005065ceb7551e80eeb4e2c24e6515234742e4
  Author: Sam Tebbs <samuel.tebbs at arm.com>
  Date:   2025-09-19 (Fri, 19 Sep 2025)

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

  Log Message:
  -----------
  remove underlying value set


Compare: https://github.com/llvm/llvm-project/compare/44f216a28c02%5E...23005065ceb7

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