search

plctlab / llvm-project

PLCT实验室的 RISC-V V Spec 实现，基于llvm/llvm-project，rkruppe/rvv-llvm 和 https://repo.hca.bsc.es/gitlab/rferrer/llvm-epi-0.8
158 stars 42 forks source link

[VP][DAGCombiner] Use `simplifySelect` when combining vp.select. #66

Closed AinsleySnow closed 7 months ago

AinsleySnow commented 7 months ago

Follow up of #79101. Adding simplifySelect to visitVP_SELECT does not result in too much modifications.

simplifySelect does not fold vselect(1, T, F) to T. Although we can check if a cond is all ones to achieve this, as a92f5a0 suggests it might be more complex, so I think it is reasonable to place related modifications in a separate PR.

ChunyuLiao commented 7 months ago

内部仓库可以合并，但是我还是觉得这种方式upstream是不接受的，但也可以去试试