[PATCH] D124450: [AMDGPU] Remove hasOneUse check from scalar select pattern

Piotr Sobczak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 09:19:19 PDT 2022


piotr added a comment.

I'd love this to go in, but when I added the `hasOneUse()` check it was certainly needed.  If my old notes serve me well there was a crash in ctlz.ll test and I concluded this check was needed to avoid some shenanigans in the si-fix-sgpr-copies. I need to double check if the issue has been fixed or is just hidden.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124450/new/

https://reviews.llvm.org/D124450



More information about the llvm-commits mailing list