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

 
 
> Непонятки с оптимизацией tipedef struct, компилятор игнорирует сложение при разных уровнях оптимизации
__nik__
сообщение May 4 2011, 10:46
Сообщение #1


Участник
*

Группа: Свой
Сообщений: 52
Регистрация: 30-11-05
Из: С-Пб
Пользователь №: 11 619



Добрый день!
у меня такая проблемка образовалась, при разных уровнях оптимизации работают по разному сложения
Name = (Typ_t __farflash*)( TablModVal|0x010000L);
замена | на + результата не приносит.

все данные арсположены выше 0х10000
есть хеадер с дефаном
#define data 0x3333
#define data2 0x4444

сам макрос
использование
//GETTABLE_PTR(адрес расположения ссылки на таблицу,имя типедеф стректуры,Name)

#define GETTABLE_PTR(TablMod,Typ_t,Name) Typ_t __farflash *Name;\
Addr16_t TablModVal;\
TablModVal = * (Addr16_t __farflash*)(TablMod|0x10000L);\
Name = (Typ_t __farflash*)( TablModVal|0x010000L); <---вот тут складывает не верно вернее берет только 16бит

в результате последняя операция |0x010000L вовсе игнорируется и копируются 16 бит данных в результате старший байт абреса становится любым. это раньше работало но вот в одном месте наткнулся на вот такой глюк. если поменять оптимизацию то |0x010000L выполняется.
как данное можно сделать чтоб не зависило от уровня оптимизации, код не поместится
Go to the top of the page
 
+Quote Post
 
Start new topic
Ответов (1 - 11)
jorikdima
сообщение May 4 2011, 13:13
Сообщение #2


тут может быть ваша реклама
*****

Группа: Свой
Сообщений: 1 164
Регистрация: 15-03-06
Из: Санкт-Петербург/CA
Пользователь №: 15 280



TablModVal вероятно 16 битный. Вот он и не осознает как можно сделать "или" с 16 битовым числом и числом более 16 бит.
Go to the top of the page
 
+Quote Post
SysRq
сообщение May 4 2011, 20:37
Сообщение #3


Чайник, 1 литр
****

Группа: Свой
Сообщений: 655
Регистрация: 17-05-06
Из: Moscow
Пользователь №: 17 168



ULL?
Go to the top of the page
 
+Quote Post
__nik__
сообщение May 5 2011, 05:43
Сообщение #4


Участник
*

Группа: Свой
Сообщений: 52
Регистрация: 30-11-05
Из: С-Пб
Пользователь №: 11 619



TablModVal замена на 32 ничего не меняет.
появилось предположение что иар оптимизирует по длинне используемых данных
Go to the top of the page
 
+Quote Post
sergeeff
сообщение May 5 2011, 06:21
Сообщение #5


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

Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007



А каков sizeof(Typ_t __farflash*)?
Go to the top of the page
 
+Quote Post
MrYuran
сообщение May 5 2011, 06:42
Сообщение #6


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Замечание по ходу:
Никогда не пишите так макросы.
Можно ненароком наколоться, например:
if(...) GETTABLE_PTR(...);

Обычно для макросов используют скобки do{...}while(0)


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
sergeeff
сообщение May 5 2011, 06:57
Сообщение #7


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

Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007



Для проверки:

Код
#define GETTABLE_PTR(TablMod,Typ_t,Name) do { Typ_t __farflash *Name;   \
Addr32_t TablModVal = (Addr32_t)TablMod;                                \
TablModVal |= 0x10000L; \
TablModVal = *( (Addr32_t __farflash*)(TablModVal)); \
TablModVal |= 0x10000L; \
Name = (Typ_t __farflash*)(TablModVal); \
} while (0)
Go to the top of the page
 
+Quote Post
MrYuran
сообщение May 5 2011, 09:03
Сообщение #8


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Цитата(sergeeff @ May 5 2011, 10:57) *
Для проверки:
...
Name = (Typ_t __farflash*)(TablModVal);
} while (0)

Одного слеша не хватает sm.gif


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
sergeeff
сообщение May 5 2011, 09:15
Сообщение #9


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

Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007



Цитата(MrYuran @ May 5 2011, 13:03) *
Одного слеша не хватает sm.gif


Потерялся при copy/paste. Поправил.
Go to the top of the page
 
+Quote Post
__nik__
сообщение May 5 2011, 13:12
Сообщение #10


Участник
*

Группа: Свой
Сообщений: 52
Регистрация: 30-11-05
Из: С-Пб
Пользователь №: 11 619



Цитата(MrYuran @ May 5 2011, 10:42) *
Замечание по ходу:
Никогда не пишите так макросы.
Можно ненароком наколоться, например:
if(...) GETTABLE_PTR(...);

Обычно для макросов используют скобки do{...}while(0)


IAR не принимает ( помоему он вообще весьма болезненно относится к {} В дефайнах
Go to the top of the page
 
+Quote Post
MrYuran
сообщение May 5 2011, 13:24
Сообщение #11


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Цитата(__nik__ @ May 5 2011, 17:12) *
IAR не принимает ( помоему он вообще весьма болезненно относится к {} В дефайнах

Не его собачье дело, обязан.
Это общепринятая методика.
Чего хоть пишет-то?

Вот, у меня работает:
#define DyrgNogoy2 do{P3OUT |= 1; P3OUT &= ~1;}while(0)
IAR для MSP430


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
__nik__
сообщение May 11 2011, 15:04
Сообщение #12


Участник
*

Группа: Свой
Сообщений: 52
Регистрация: 30-11-05
Из: С-Пб
Пользователь №: 11 619



Цитата(MrYuran @ May 5 2011, 17:24) *
Не его собачье дело, обязан.
Это общепринятая методика.
Чего хоть пишет-то?

Вот, у меня работает:
#define DyrgNogoy2 do{P3OUT |= 1; P3OUT &= ~1;}while(0)
IAR для MSP430

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

void activate_out(uchar number)
{
GETTABLE_PTR(CfgSFBModul,SFBSetings_t,SFBSet);
if(number > 2) return;
number--;
outs_state2bus[number].norm=0; outs_state2bus[number].active=1;
if(SFBSet->outs_flags[number]&0x01) // вот тут адрес SFBSet = 0х003BCE
{
switch(number)
...

void deactivate_out(uchar number)
{
GETTABLE_PTR(CfgSFBModul,SFBSetings_t,SFBSet);
if(number > 2) return;
number--;
outs_state2bus[number].norm=1; outs_state2bus[number].active=0;
if(SFBSet->outs_flags[number]&0x01) // вот тут адрес SFBSet = 0х013BCE как и должен быть
{
switch(number)
{
...
после многоточий код совпадает на 100% еще раз наткнулся на теже грабли в другом месте,
Go to the top of the page
 
+Quote Post

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

 


RSS Текстовая версия Сейчас: 25th July 2025 - 12:28
Рейтинг@Mail.ru


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