Index
> Main > idiv -- drops an application down |
| Author |
|
|
Overclick 10 Jun 2021, 23:44
Ups, wrong place I post it ))
|
|||
10 Jun 2021, 23:44 |
|
|
revolution 11 Jun 2021, 00:03
We don't know what you have done. Show some code.
|
|||
|
11 Jun 2021, 00:03 |
|
|
Walter 11 Jun 2021, 01:18
Isn't it usual to use the "cdq" instruction immediately before the idiv?
|
|||
|
11 Jun 2021, 01:18 |
|
|
revolution 11 Jun 2021, 01:46
There are two types of idiv instruction. Which is why I asked to see the code so we can see what is being done.
I expect the error reported is division by zero, but the OP doesn't say, so we can't know for sure. Division by zero is generated whenever the result can't fit into the destination. So it happens for more cases than just a zero divisor. |
|||
|
11 Jun 2021, 01:46 |
|
|
Overclick 11 Jun 2021, 03:08
I experimenting with some audio filters and forgot to clear rdx, that how I found it. There is non Zero for sure. Anyway, why OS deals with it?
|
||||||||||
|
11 Jun 2021, 03:08 |
|
|||||||||
|
Overclick 11 Jun 2021, 03:33
Actually there have to be pair of idiv r12 idiv r13, I tried to optimise them to one divide. Anyway this filter is too slowly for me and I have better idea.
|
|||
|
11 Jun 2021, 03:33 |
|
|
revolution 11 Jun 2021, 03:47
You are using the single operand version of idiv. That means it takes rdx and rax as the numerator.
What is your error? It is division by zero, right? You forgot to say. If the absolute value of rdx is greater than or equal to r12 then you will get a DivZero error. The result would be greater than 64 bits. For example if we assume that each register is a single decimal digit: 30 / 3 needs two digits for the result and can't fit a single register. |
|||
|
11 Jun 2021, 03:47 |
|
|
Overclick 11 Jun 2021, 03:55
Is that means "division by zero" where result doesn't feet to single register? Interesting...
Why OS then? |
|||
|
11 Jun 2021, 03:55 |
|
|
revolution 11 Jun 2021, 04:05
The OS receives all CPU generated traps. It is how the CPU works.
|
|||
|
11 Jun 2021, 04:05 |
|
|
Overclick 11 Jun 2021, 04:06
Where to read about another implementation of idiv? I know only one. Don't you mistake with imul instruction?
|
|||
|
11 Jun 2021, 04:06 |
|
|
Overclick 11 Jun 2021, 06:59
Quote:
You right, cqo for rdx |
|||
|
11 Jun 2021, 06:59 |
|
|
Furs 11 Jun 2021, 12:50
Overclick wrote: If I don't clear rdx before idiv, the next command will never run. Not clearing rdx means you have garbage values there. Even if it didn't crash, your results would be totally wrong. |
|||
|
11 Jun 2021, 12:50 |
|
|
Overclick 11 Jun 2021, 15:43
Quote:
As I said I forgot to do that. Just interested why it can't be processed by developer as usual flag changing operation. |
|||
|
11 Jun 2021, 15:43 |
|
|
revolution 12 Jun 2021, 00:47
Overclick wrote: Just interested why it can't be processed by developer as usual flag changing operation. If you use the FPU then you can set the control register to generate infinity. But no such equivalent exists for the integer core. If you need to avoid the trap then you can compare rdx to the denominator and skip the division. Code: ; for positive numbers cmp rdx, r12 jae .too_large idiv r12 ;... .too_large: ;... |
|||
|
12 Jun 2021, 00:47 |
|
|
Overclick 12 Jun 2021, 04:47
Too slowly all of that. Sadly there is not much options to divide integer, no one for SSE. Only bit shifting can help a little. I return to float, it working slower but need less operations to realise all I want. Classical FPU is absolutely slowmo. 10 times slower than integer when SSE just twice for single dword. Fully packed is ok.
|
|||
|
12 Jun 2021, 04:47 |
|
|
FlierMate 12 Jun 2021, 12:53
I noticed the same (in 32-bit):
Code: 0: A1 04 10 40 00 MOV EAX,DS:0x401004 5: 31 D2 XOR EDX,EDX 7: BB 02 00 00 00 MOV EBX,0X2 C: F7 FB IDIV EBX E: A3 04 10 40 00 MOV DS:0x401004,EAX Generated for my own language: Code: LET b= 20 b /= 2 Have to clear EDX. Thanks to this thread, now I understand far better. |
|||
|
12 Jun 2021, 12:53 |
|
|
revolution 12 Jun 2021, 12:57
If you use IDIV then you might want to consider using CDQ instead of XOR EDX,EDX
Or alternatively if EAX is unsigned, use DIV, and keep the XOR EDX,EDX |
|||
|
12 Jun 2021, 12:57 |
|
< Last Thread | Next Thread > |
Forum Rules:
|