[all-commits] [llvm/llvm-project] 29b8b7: [LV] Move check if any vector insts will be genera...

[Florian Hahn via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 29b8b72117845dc3d20e70fbfa85c590fa9c7830
      https://github.com/llvm/llvm-project/commit/29b8b72117845dc3d20e70fbfa85c590fa9c7830
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2024-07-07 (Sun, 07 Jul 2024)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
    M llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
    M llvm/lib/Transforms/Vectorize/VPlanAnalysis.cpp
    M llvm/test/Transforms/LoopVectorize/SystemZ/zero_unroll.ll
    M llvm/test/Transforms/LoopVectorize/X86/induction-costs.ll
    M llvm/test/Transforms/LoopVectorize/pr32859.ll
    M llvm/test/Transforms/LoopVectorize/vplan-incomplete-cases.ll

  Log Message:
  -----------
  [LV] Move check if any vector insts will be generated to VPlan. (#96622)

This patch moves the check if any vector instructions will be generated
from getInstructionCost to be based on VPlan. This simplifies
getInstructionCost, is more accurate as we check the final result and
also allows us to exit early once we visit a recipe that generates
vector instructions.

The helper can then be re-used by the VPlan-based cost model to match
the legacy selectVectorizationFactor behavior, this fixing a crash and
paving the way to recommit
https://github.com/llvm/llvm-project/pull/92555.

PR: https://github.com/llvm/llvm-project/pull/96622



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