[all-commits] [llvm/llvm-project] d6b993: [AArch64][llvm] Preserve FP_TO_*_SAT VT operand in...

Jonathan Thackray via All-commits all-commits at lists.llvm.org
Wed Feb 11 04:48:05 PST 2026


  Branch: refs/heads/users/jthackray/with-precommit
  Home:   https://github.com/llvm/llvm-project
  Commit: d6b9930fdacbc7b502879c9132713aac2ff28285
      https://github.com/llvm/llvm-project/commit/d6b9930fdacbc7b502879c9132713aac2ff28285
  Author: Jonathan Thackray <jonathan.thackray at arm.com>
  Date:   2026-02-11 (Wed, 11 Feb 2026)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    M llvm/test/CodeGen/AArch64/arm64-cvt-simd-fptoi.ll
    M llvm/test/CodeGen/AArch64/arm64-cvtf-simd-itofp.ll

  Log Message:
  -----------
  [AArch64][llvm] Preserve FP_TO_*_SAT VT operand in SVE scalar-combine

Updated RUN lines and generated new `CHECK‑SME`/`CHECK‑SVE` lines in:

    llvm/test/CodeGen/AArch64/arm64-cvt-simd-fptoi.ll
    llvm/test/CodeGen/AArch64/arm64-cvtf-simd-itofp.ll

by adding `-force-streaming` and `-force-streaming-compatible` runs,
as pre-commit tests for a future change to enable FPRCVT streaming.

This triggers a SVE scalar-combine path which requires a code update.
FP_TO_*_SAT nodes require operand 1 (the saturation VT) to be present.
Without it the node is malformed and hits the SelectionDAG assertion
“Invalid child # of SDNode!”.

Also, skip the SVE combine if the scalar/custom op is already legal.



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