[llvm] [DXIL] Add lowering for `reversebits` and `trunc` (PR #86909)

Helena Kotas via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 28 12:04:28 PDT 2024


hekota wrote:

> might be good to have a reverversebits error case so we know dxil doesn't get generated for floats.

I assumed we are only adding lowering error cases when the llvm intrinsic supports more types than DXIL. For example in the `cos_error.ll` there is error case for the `double` overload, but not for any integer overloads because `llvm.cos` does not support integers. The `llvm.bitreverse` intrinsics only has only integer overloads and `opt` fails to even load the code with `llvm.bitreverse.f32`, let alone reach the `dxil-op-lower` pass.

https://github.com/llvm/llvm-project/pull/86909


More information about the llvm-commits mailing list