[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

jianghc724 wrote: should be cvts_biasph_hf8 :) https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

https://github.com/phoebewang edited https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

https://github.com/e-kud approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

phoebewang wrote: > should be cvts_biasph_hf8 :) Good catch! Proves moving the s ahead is wisdom :) https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

phoebewang wrote: > > > > Went through the patch with my eyes, I'm not sure how useful it was but > > > > wasn't able to spot anything. > > > > > Here is GCC patch: > > > > > https://gcc.gnu.org/pipermail/gcc-patches/2025-March/678227.html > > > > > > > > > > > > I haven't found any changes r

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

Rejur wrote: > > > Went through the patch with my eyes, I'm not sure how useful it was but > > > wasn't able to spot anything. > > > > Here is GCC patch: > > > > https://gcc.gnu.org/pipermail/gcc-patches/2025-March/678227.html > > > > > > > > > I haven't found any changes regarding `vcvt2ph2`

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

jianghc724 wrote: > > Went through the patch with my eyes, I'm not sure how useful it was but > > wasn't able to spot anything. > > > Here is GCC patch: > > > https://gcc.gnu.org/pipermail/gcc-patches/2025-March/678227.html > > > > > > I haven't found any changes regarding `vcvt2ph2` and no o

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

phoebewang wrote: > Went through the patch with my eyes, I'm not sure how useful it was but > wasn't able to spot anything. > > > Here is GCC patch: > > https://gcc.gnu.org/pipermail/gcc-patches/2025-March/678227.html > > I haven't found any changes regarding `vcvt2ph2` and no other similar

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

phoebewang wrote: Here is GCC patch: https://gcc.gnu.org/pipermail/gcc-patches/2025-March/678227.html https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listi

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

https://github.com/phoebewang ready_for_review https://github.com/llvm/llvm-project/pull/131592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Phoebe Wang (phoebewang) Changes - Add '_' after cvt[t]s intrinsics when 's' is for saturation; - Add 's_' for all ipcvt[t] intrinsics since they are all saturation ones; This is to solve potential confusion since 's' before a type usually

[clang] [X86][AVX10.2] Use 's_' for saturate-convert intrinsics (PR #131592)

llvmbot wrote: @llvm/pr-subscribers-backend-x86 Author: Phoebe Wang (phoebewang) Changes - Add '_' after cvt[t]s intrinsics when 's' is for saturation; - Add 's_' for all ipcvt[t] intrinsics since they are all saturation ones; This is to solve potential confusion since 's' before a type u