[all-commits] [llvm/llvm-project] 8ee685: [NFC][DirectX] fix intrinsics that need IntrNoMem ...

[Farzon Lotfi via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8ee685e601617031f50daa3df5e0f1aa38e15108
      https://github.com/llvm/llvm-project/commit/8ee685e601617031f50daa3df5e0f1aa38e15108
  Author: Farzon Lotfi <1802579+farzonl at users.noreply.github.com>
  Date:   2024-09-16 (Mon, 16 Sep 2024)

  Changed paths:
    M llvm/include/llvm/IR/IntrinsicsDirectX.td
    M llvm/test/CodeGen/DirectX/reversebits.ll

  Log Message:
  -----------
  [NFC][DirectX] fix intrinsics that need IntrNoMem and test typo (#108852)

In the process of adding scalarization support for DirectX target
intrinsics I found that intrinsics that weren't marked with `IntrNoMem`
did not get removed by
`RecursivelyDeleteTriviallyDeadInstructionsPermissive`. So this change
is to make it more clear that our intrinsics don't have side effects.

I only added `IntrNoMem` to the intrinics in `IntrinsicsDirectX.td` I
was involved with. There a potentially a few other cases that might
warrant this attribute, but will need input on the others.



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