[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