[all-commits] [llvm/llvm-project] da8d7f: [RISCV] Unify non-vp and vp rounding intrinsic cos...

[Luke Lau via All-commits]

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

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
    M llvm/test/Analysis/CostModel/RISCV/fround.ll

  Log Message:
  -----------
  [RISCV] Unify non-vp and vp rounding intrinsic costing (#147872)

Currently we have slightly different costing for the vp and non-vp
version of the rounding intrinsics.

We can delete this code and use the generic BasicTTIImpl code for the vp
intrinsics which falls back to the non-vp versions.

I'm not sure if the zvfh costing is correct, this should probably be
fixed in a follow up patch. At the moment the non-vp cost is more
important since it is what the loop vectorizer will use.



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