[all-commits] [llvm/llvm-project] a5e129: [CostModel][X86] getVectorInstrCost - correctly co...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a5e129ccdedf5c269a8e0fcad5e21381a7f0342c
      https://github.com/llvm/llvm-project/commit/a5e129ccdedf5c269a8e0fcad5e21381a7f0342c
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2025-01-07 (Tue, 07 Jan 2025)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/lib/Transforms/Vectorize/VectorCombine.cpp
    M llvm/test/Analysis/CostModel/X86/vector-insert-value.ll

  Log Message:
  -----------
  [CostModel][X86] getVectorInstrCost - correctly cost v4f32 insertelement into index 0

This is just the MOVSS instruction (SSE41 INSERTPS is still necessary for index != 0)

This exposed an issue in VectorCombine::foldInsExtFNeg - we need to use the more general SK_PermuteTwoSrc shuffle kind to allow getShuffleCost to match other shuffle kinds (not just SK_Select).



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