dynarmic icon indicating copy to clipboard operation
dynarmic copied to clipboard

decoder/arm: Make CLZ required encoding bits more relaxed

Open pent0 opened this issue 3 years ago • 2 comments

The official document marks these now ignored bits (using brackets) as should be there, but not necessary. I was encountering a JIT generate this instruction running fine on a real CPU.

image1 image2

pent0 avatar Nov 25 '21 10:11 pent0

Note to self: Bits 8-11 and 16-19 are SBO.

TODO: This breaks tests because unicorn doesn't recognise these encodings.

merryhime avatar Nov 27 '21 17:11 merryhime

This also applies to QADD, QSUB, QDADD, QDSUB, I will keep it to myself for now (encountered it, though I don't want to complicate things).

pent0 avatar Nov 28 '21 02:11 pent0

I think It would be better if I modify this in a fork for my need. Dynarmic should stick to normal standard, and I think trying to get the tests to pass maybe a hassle too!

pent0 avatar Dec 24 '22 16:12 pent0