Телеграмм чат группы prographon страница 8455

насколько хорошо может соптимизтроваться такой код?

ovl.Offset += transfered;
if (ovl.Offset < transfered) ++ovl.OffsetHigh;

Поле OffsetHigh располагается сразу за Offset в одной структуре

00:13пожаловаться #1

disba1ancer

насколько хорошо может соптимизтроваться такой код?

ovl.Offset += transfered;
if (ovl.Offset < transfered) ++ovl.OffsetHigh;

Поле OffsetHigh располагается сразу за Offset в одной структуре

Ты уверен, что эти две строки стоят твоего времени оптимизации?)
Ты же рядом в файл ходишь

00:29пожаловаться #2

Ты уверен, что эти две строки стоят твоего времени оптимизации?)
Ты же рядом в файл ходишь

да чисто из интереса

00:29пожаловаться #3

но вообще компиль соптимизировал if до простого условного джампа

00:29пожаловаться #4

Ну они тупят тут

Pavel Kazakov in pro.graphon (and gamedev)

00:30пожаловаться #5

disba1ancer

но вообще компиль соптимизировал if до простого условного джампа

это как? lea, а потом jmp?

00:30пожаловаться #6

Но не надо это оптимизировать пока не видишь хоть сколько-то в профайлере, вредно для твоего проекта

00:30пожаловаться #7

Но в целом
ovl.OffsetHigh += (ovl.Offset < transfered);
лучше генерит код

00:30пожаловаться #8

Pavel Kazakov

это как? lea, а потом jmp?

джамп по переносу

Pavel Kazakov in pro.graphon (and gamedev)

00:30пожаловаться #9

если там cmp с вариациями, то это просто if и есть

00:31пожаловаться #10

Pavel Kazakov

если там cmp с вариациями, то это просто if и есть

там нет cmp

00:31пожаловаться #11

проверяется перенос предыдущего add

Pavel Kazakov in pro.graphon (and gamedev)

00:31пожаловаться #12

Но не надо это оптимизировать пока не видишь хоть сколько-то в профайлере, вредно для твоего проекта

вот тут соглашусь)

00:31пожаловаться #13

ну в общем вот:

//ovl.Offset += transfered;
mov    %eax,%edx
add    0x10(%rbx),%edx
mov    %edx,0x10(%rbx)
//if (ovl.Offset < transfered) ++ovl.OffsetHigh;
jae    c24 <_ZN3dse2os2io14IOTarget_win324readEPcy+0x44>
addl   $0x1,0x14(%rbx)

00:32пожаловаться #14

Но в целом
ovl.OffsetHigh += (ovl.Offset < transfered);
лучше генерит код

00:33пожаловаться #15

Сделай так, там add + adc, это выглядит максимально адекватно

00:33пожаловаться #16

Сделай так, там add + adc, это выглядит максимально адекватно

да, но тут можно вообще оба offset поля расценивать как одно большое 64битное целое и делать соответствующие действия, но, возможно, компилятор это видит и считает что это учитывать не нужно

00:36пожаловаться #17

Не видит, они тупят что-то (даже с правильным выравниванием)

00:37пожаловаться #18

Но забей, иди логику пиши, это не стоит того

00:37пожаловаться #19

Но забей, иди логику пиши, это не стоит того

мне просто интересно было, сможет компиль здесь или нет, вручную править я, разумеется, ничего не буду