> Increased FAILS are LMUL = M4. I have analyzed the codegen. Looks > reasonable. > > Moreover, When I removed 'popcount_64' and test, all passed no matter > apply this patch or not. > > I think it is because popcount64 is buggy in RV32, this patch trigger > LMUL = 4 bug already existed that we were lucky. > > So I suggest this patch should go ahead and ignore popcount issue for > now. (I will send V3 with fixing dump FAILs). > > I am not familiar with popcount, Robin. Any suggestions? Yeah, agree. popcount_64 might be wrong and it's unlikely that your patch causes it. Will have a look.
Regards Robin