[all-commits] [llvm/llvm-project] df12ba: [libclc] Use CLC conversion builtins in CLC functi...

[Fraser Cormack via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: df12bad075b77953fc3af6fc3db3b7dd25a81977
      https://github.com/llvm/llvm-project/commit/df12bad075b77953fc3af6fc3db3b7dd25a81977
  Author: Fraser Cormack <fraser at codeplay.com>
  Date:   2025-02-18 (Tue, 18 Feb 2025)

  Changed paths:
    M libclc/clc/include/clc/clc_convert.h
    M libclc/clc/lib/generic/integer/clc_mad_sat.cl
    M libclc/clc/lib/generic/integer/clc_mul_hi.cl
    M libclc/clc/lib/generic/integer/clc_upsample.cl
    M libclc/generic/lib/math/clc_exp10.cl
    M libclc/generic/lib/math/clc_fmod.cl
    M libclc/generic/lib/math/clc_pow.cl
    M libclc/generic/lib/math/clc_pown.cl
    M libclc/generic/lib/math/clc_powr.cl
    M libclc/generic/lib/math/clc_remainder.cl
    M libclc/generic/lib/math/clc_remquo.cl
    M libclc/generic/lib/math/clc_rootn.cl

  Log Message:
  -----------
  [libclc] Use CLC conversion builtins in CLC functions (#127628)

This commit is a broad update across libclc to use the CLC conversion
builtins in CLC functions, even those with a '__clc' prefix in the
generic folder. This better prepares them for an official move to the
CLC library in time.

The CLC conversion builtins have an additional benefit in that they
support scalars, unlike the __builtin_convertvector builtin which we
were using previously. This allows us to simplify some shared
definitions.

There is one change to the IR, in the scalar upsample(char, uchar)
builtin. It now sign-extends the first argument to i16, where before it
zero-extended it. This appears to be correct, and matches the vector
behaviour.



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