[all-commits] [llvm/llvm-project] 7eb14d: [TTI] Fix value-based BasicTTIImpl vp.{gather, scat...

[Luke Lau via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7eb14d9dd19524719c831ecf75d5a8df735db49d
      https://github.com/llvm/llvm-project/commit/7eb14d9dd19524719c831ecf75d5a8df735db49d
  Author: Luke Lau <luke at igalia.com>
  Date:   2025-07-11 (Fri, 11 Jul 2025)

  Changed paths:
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/test/Analysis/CostModel/RISCV/vp-intrinsics.ll

  Log Message:
  -----------
  [TTI] Fix value-based BasicTTIImpl vp.{gather,scatter} costing (#148020)

After #147677 we now preserve value based costing for vp intrinsics
instead of switching it to type based costing.

However for vp.gather and vp.scatter, even though they fallback to their
functionally equivalent masked.gather and masked.scatter, the number of
arguments are different due to the alignment being a dedicated argument.

This caused a crash detected at
https://lab.llvm.org/staging/#/builders/210/builds/988

Thix fixes it by explicitly handling the two intrinsics and adding test
coverage.

Note that the type based costing isn't yet implemented for
masked.gather/masked.scatter so it doesn't show up correctly.



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