реклама на сайте
подробности

 
 
 
Reply to this topicStart new topic
> Длинные операнды, Выполняются с ошибками
FormatCft
сообщение Apr 27 2009, 11:48
Сообщение #1


Местный
***

Группа: Свой
Сообщений: 331
Регистрация: 22-07-08
Из: Р О С С И Я
Пользователь №: 39 145



Доброго времени суток.

GCC использую.
Обнаружил, что операнд типа такого:
Код
a = a + (((b << 4) + rol (key[(sum % 4) + 4], b)) ^ (d + sum) ^ ((b >> 5) + rol (key[sum % 4], b >> 27)));
#define rol(N, R) (N<<R)|(N>>(32-R))

выполняется с ошибками.
Попытался разкидать это выражение по отдельным, всё стало вроде в норме.
Может я чего-то недопонимаю. Откуда всё-таки ошибки?

Спасибо!
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Apr 27 2009, 11:53
Сообщение #2


Гуру
******

Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448



Цитата(FormatCft @ Apr 27 2009, 15:48) *
Откуда всё-таки ошибки?

Наверное, с макросами внимательнее нужно быть:
Код
#define rol(N, R) ((N<<R)|(N>>(32-R)))
Go to the top of the page
 
+Quote Post
Rst7
сообщение Apr 27 2009, 11:56
Сообщение #3


Йа моск ;)
******

Группа: Модераторы
Сообщений: 4 345
Регистрация: 7-07-05
Из: Kharkiv-city
Пользователь №: 6 610



Цитата
Наверное, с макросами внимательнее нужно быть:


Тогда уже так:
Код
#define rol(N, R) (((N)<<(R))|((N)>>(32-(R))))


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


--------------------
"Практика выше (теоретического) познания, ибо она имеет не только достоинство всеобщности, но и непосредственной действительности." - В.И. Ленин
Go to the top of the page
 
+Quote Post
meister
сообщение Apr 27 2009, 12:00
Сообщение #4


Местный
***

Группа: Участник
Сообщений: 219
Регистрация: 20-11-07
Пользователь №: 32 484



Код
unsigned long rol(unsigned long n, unsigned r) { return (n << r) | (n >> (32 - r)); }
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Apr 27 2009, 12:01
Сообщение #5


Гуру
******

Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448



Цитата(Rst7 @ Apr 27 2009, 15:56) *
Тогда уже так

Таки да, b >> 27 упустил из вида.
Go to the top of the page
 
+Quote Post
Rst7
сообщение Apr 27 2009, 12:02
Сообщение #6


Йа моск ;)
******

Группа: Модераторы
Сообщений: 4 345
Регистрация: 7-07-05
Из: Kharkiv-city
Пользователь №: 6 610



Цитата
unsigned long rol(unsigned long n, unsigned r) { return (n << r) | (n >> (32 - r)); }


А смысл? Тогда еще надо и inline forced написать.


--------------------
"Практика выше (теоретического) познания, ибо она имеет не только достоинство всеобщности, но и непосредственной действительности." - В.И. Ленин
Go to the top of the page
 
+Quote Post
FormatCft
сообщение Apr 27 2009, 12:45
Сообщение #7


Местный
***

Группа: Свой
Сообщений: 331
Регистрация: 22-07-08
Из: Р О С С И Я
Пользователь №: 39 145



Блин!!! Я так вам благодарен!!! СПАСИБО!!!
Go to the top of the page
 
+Quote Post
DpInRock
сообщение Apr 27 2009, 13:57
Сообщение #8


Гуру
******

Группа: Участник
Сообщений: 2 254
Регистрация: 4-05-07
Из: Moscow
Пользователь №: 27 515



За что следует смертельно любить Си, так это за неумеренное количество скобок.


--------------------
On the road again (Canned Heat)
Go to the top of the page
 
+Quote Post
mdmitry
сообщение Apr 27 2009, 14:24
Сообщение #9


Начинающий профессионал
*****

Группа: Свой
Сообщений: 1 215
Регистрация: 25-10-06
Из: СПб
Пользователь №: 21 648



Цитата(DpInRock @ Apr 27 2009, 17:57) *
За что следует смертельно любить Си, так это за неумеренное количество скобок.

При определении #define crying.gif


--------------------
Наука изощряет ум; ученье вострит память. Козьма Прутков
Go to the top of the page
 
+Quote Post
SSerge
сообщение Apr 27 2009, 15:34
Сообщение #10


Профессионал
*****

Группа: Свой
Сообщений: 1 719
Регистрация: 13-09-05
Из: Novosibirsk
Пользователь №: 8 528



Цитата(DpInRock @ Apr 27 2009, 20:57) *
За что следует смертельно любить Си, так это за неумеренное количество скобок.

Расскажите эту шутку любителям Lisp-а.


--------------------
Russia est omnis divisa in partes octo.
Go to the top of the page
 
+Quote Post
meister
сообщение May 7 2009, 14:11
Сообщение #11


Местный
***

Группа: Участник
Сообщений: 219
Регистрация: 20-11-07
Пользователь №: 32 484



Цитата(Rst7 @ Apr 27 2009, 16:02) *
А смысл?


Дабы избежать удивительные эффекты от подстановки в качестве параметров макросов выражений.

Цитата(Rst7 @ Apr 27 2009, 16:02) *
Тогда еще надо и inline forced написать.


Оставьте оптимизитору его работу.
Go to the top of the page
 
+Quote Post
Rst7
сообщение May 7 2009, 16:32
Сообщение #12


Йа моск ;)
******

Группа: Модераторы
Сообщений: 4 345
Регистрация: 7-07-05
Из: Kharkiv-city
Пользователь №: 6 610



Цитата
Дабы избежать удивительные эффекты от подстановки в качестве параметров макросов выражений.


Этот вопрос решается скобками.

Цитата
Оставьте оптимизитору его работу.


Да ну? Тут принципиально автору хочется строгого инлайна. Значит - надо писать, а не надеяться, что компиллер заинлайнит.


--------------------
"Практика выше (теоретического) познания, ибо она имеет не только достоинство всеобщности, но и непосредственной действительности." - В.И. Ленин
Go to the top of the page
 
+Quote Post
meister
сообщение May 7 2009, 19:50
Сообщение #13


Местный
***

Группа: Участник
Сообщений: 219
Регистрация: 20-11-07
Пользователь №: 32 484



Цитата(Rst7 @ May 7 2009, 20:32) *
Этот вопрос решается скобками.


Код
#define MAX(A, B) (((A) > (B)) ? (A) : (B))

int uberlongtimefunction(int x)
{
   ...
}

int main(void)
{
    int a = 7;
    int b = 19;
    int m = MAX(++a, uberlongtimefunction(b));
}


Сообщение отредактировал meister - May 7 2009, 20:03
Go to the top of the page
 
+Quote Post
xemul
сообщение May 8 2009, 10:33
Сообщение #14



*****

Группа: Свой
Сообщений: 1 928
Регистрация: 11-07-06
Пользователь №: 18 731



Цитата(meister @ May 7 2009, 23:50) *
Код
#define MAX(A, B) (((A) > (B)) ? (A) : (B))
...
int main(void)
{
    int m = MAX(++a, uberlongtimefunction(b));
}

О такой беде можно прочесть еще в первом издании Кернигана и Ричи. Если человеку приспичит походить по граблям с полированными ручками, его ни книжки, ни скобки не остановят.
Rst7, имхо, говорил несколько о другом. А компиляторы под ПИК'и от ХТсофт, н-р, пользовательские функции инлайнить просто отказываются.
Go to the top of the page
 
+Quote Post
Rst7
сообщение May 8 2009, 19:21
Сообщение #15


Йа моск ;)
******

Группа: Модераторы
Сообщений: 4 345
Регистрация: 7-07-05
Из: Kharkiv-city
Пользователь №: 6 610



Цитата
Rst7, имхо, говорил несколько о другом.


Безусловно. Для криптографии нужен строгий инлайн, иначе будет медленно. Вне зависимости от того, что решит компилятор - инлайнить или нет. Значит, нужна принудительная прагма. Так нафига столько лишней писанины?

Цитата
А компиляторы под ПИК'и от ХТсофт, н-р, пользовательские функции инлайнить просто отказываются.


Разговор об ARM, если мне не изменяет память (по недавнему вопросу топикстартера в другой ветке). И о гнусе.


--------------------
"Практика выше (теоретического) познания, ибо она имеет не только достоинство всеобщности, но и непосредственной действительности." - В.И. Ленин
Go to the top of the page
 
+Quote Post

Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 21st June 2025 - 05:42
Рейтинг@Mail.ru


Страница сгенерированна за 0.01492 секунд с 7
ELECTRONIX ©2004-2016