[all-commits] [llvm/llvm-project] 37ed2e: [Scalarizer] Make `*_with_overflow` intrinsics sca...

[Deric Cheung via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 37ed2e6b33882cdf5dcc640f59965c5c522b489e
      https://github.com/llvm/llvm-project/commit/37ed2e6b33882cdf5dcc640f59965c5c522b489e
  Author: Deric Cheung <cheung.deric at gmail.com>
  Date:   2025-02-13 (Thu, 13 Feb 2025)

  Changed paths:
    M llvm/lib/Analysis/VectorUtils.cpp
    A llvm/test/Transforms/Scalarizer/sadd_with_overflow.ll
    A llvm/test/Transforms/Scalarizer/sincos.ll
    A llvm/test/Transforms/Scalarizer/smul_with_overflow.ll
    A llvm/test/Transforms/Scalarizer/ssub_with_overflow.ll
    R llvm/test/Transforms/Scalarizer/uadd_overflow.ll
    A llvm/test/Transforms/Scalarizer/uadd_with_overflow.ll
    A llvm/test/Transforms/Scalarizer/umul_with_overflow.ll
    A llvm/test/Transforms/Scalarizer/usub_with_overflow.ll

  Log Message:
  -----------
  [Scalarizer] Make `*_with_overflow` intrinsics scalarizable (#126815)

Addresses issue #126809

- Made `uadd_with_overflow`, `sadd_with_overflow`, `usub_with_overflow`,
`ssub_with_overflow`, `umul_with_overflow`, and `smul_with_overflow`
trivially scalarizable in `isTriviallyScalarizable()` from
`VectorUtils.cpp`
- Renamed and updated the test `Scalarizer/uadd_overflow.ll` to
`Scalarizer/uadd_with_overflow.ll` to check that `uadd_with_overflow`
gets scalarized
- Added a test `Scalarizer/sincos.ll` to ensure the bug fix #113625
still works



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