[all-commits] [llvm/llvm-project] 0ee8f6: [VectorCombine] Fix invalid shuffle cost argument ...

hanbeom via All-commits all-commits at lists.llvm.org
Fri Mar 7 08:40:49 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0ee8f699780569d7a6b94d61cd833285bb67eca1
      https://github.com/llvm/llvm-project/commit/0ee8f699780569d7a6b94d61cd833285bb67eca1
  Author: hanbeom <kese111 at gmail.com>
  Date:   2025-03-07 (Fri, 07 Mar 2025)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/VectorCombine.cpp
    M llvm/test/Transforms/PhaseOrdering/X86/blendv-select.ll
    M llvm/test/Transforms/VectorCombine/AArch64/shuffletoidentity.ll
    M llvm/test/Transforms/VectorCombine/X86/shuffle-of-selects.ll

  Log Message:
  -----------
  [VectorCombine] Fix invalid shuffle cost argument of foldShuffleOfSelects (#130281)

In the previous code (#128032), it specified the destination vector as the
getShuffleCost argument. Because the shuffle mask specifies the indices
of the two vectors specified as elements, the maximum value is twice the
size of the source vector. This causes a problem if the destination
vector is smaller than the source vector and specify an index in the
mask that exceeds the size of the destination vector.

Fix the problem by correcting the previous code, which was using wrong
argument in the Cost calculation.

Fixes #130250



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


More information about the All-commits mailing list