Zce Branch: BEQI and BNEI should not be inferred when Immediate is 0

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

159 stars 40 forks source link

Zce Branch: BEQI and BNEI should not be inferred when Immediate is 0 #33

Closed abukharmeh closed 2 years ago

abukharmeh commented 2 years ago

BEQI and BNEI should not be inferred when the Immediate is 0, since we could already use c.beqz and c.bnez (16 bit instructions).

ChunyuLiao commented 2 years ago

commit should fix this problem. you can try again.

abukharmeh commented 2 years ago

Hi @ChunyuLiao,

Thank you for fixing this one.

I was checking this against our proof of concept core, and noticed that the encoding is not matching the 0.50.1 release, so I could not check that everything is completely correct until #34 is also fixed.

Kind regards, Ibrahim.

Xinlong-Wu commented 2 years ago

I think we have solved the problem, so I am going to close this issue. if there is still a problem, please reopen it