Support arbitrary signal length in `count_leading_zeros`

kuznia-rdzeni / coreblocks

RISC-V out-of-order core for education and research purposes

https://kuznia-rdzeni.github.io/coreblocks/

BSD 3-Clause "New" or "Revised" License

38 stars 16 forks source link

Support arbitrary signal length in `count_leading_zeros` #751

Open tilk opened 3 weeks ago

tilk commented 3 weeks ago

Currently this function requires power of 2 length.

piotro888 commented 3 weeks ago

update PriorityEncoder to use more efficient count_leading_zeros instead of m.If loop
refactor FPURounding to use new count_leading_zeros

Hazardu commented 1 week ago

I will open a PR solving the issue sometime this week.

@Durchbruchswagen would you like the PR to target #741 instead of master branch to also do things like removing nearestpow2 that you had to add there?

Durchbruchswagen commented 1 week ago

I will open a PR solving the issue sometime this week.

@Durchbruchswagen would you like the PR to target #741 instead of master branch to also do things like removing nearestpow2 that you had to add there?

If this isn't too big of an issue for you, then sure, go ahead and target #741.