Модераторы: PILOT, ManiaK, Mazzi
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> [avr-gcc] Как побороть лишнюю оптимизацию? 
:(
    Опции темы
UniBomb
Дата 4.2.2009, 16:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок
***
Награды: 1



Профиль
Группа: Завсегдатай
Сообщений: 1754
Регистрация: 24.10.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 97
Добрый день. Столкнулся вот с какой незадачей - компиляция программы происходит так, что часть одной функции как бы выбрасывается из программы. Ошибок в функции нет, но что там говорить, вот эта функция:

(функция "как есть", в целях понимания ничего не убирал, надеюсь поймёте)
Код
void processing_data(void)
{
 if(mdns.mdfl[U] <= mdns.mdfl[PGSU1])
  { mdns.mdfl[Conc] = mdns.mdfl[A1]*mdns.mdfl[U] + mdns.mdfl[B1]; }
 else
  { mdns.mdfl[Conc] = mdns.mdfl[A2]*mdns.mdfl[U] + mdns.mdfl[B2]; }
  
 char _s_disrec = 0;
 if(mdns.mdfl[Conc] < 0) { mdns.mdfl[Conc] = 0; }
 if(mdns.mdfl[Conc] >= mdns.mdfl[Pr])
  { pereg(); }
 else if(mdns.mdfl[Conc] >= /*mdns.mdfl[P2]*/0.88)
  {
   _s_disrec = 1;
   if((flags&0x03 == 0) || (flags&0x03 == 1)) //если нет порогов или только первый
    {
     mdns.mdui[P1Count]++;
     mdns.mdui[P2Count]++;  
     WriteByteIntoExtEEPROM(0X3E, mdns.mdWorkSpace[0X46]); 
     WriteByteIntoExtEEPROM(0X3F, mdns.mdWorkSpace[0X47]);
     WriteByteIntoExtEEPROM(0X40, mdns.mdWorkSpace[0X48]); 
     WriteByteIntoExtEEPROM(0X41, mdns.mdWorkSpace[0X49]);     
     modbus_buf[9] = mdns.mdWorkSpace[0x48];//p2
     modbus_buf[8] = mdns.mdWorkSpace[0x49];//p2     
     modbus_buf[7] = mdns.mdWorkSpace[0x46];//p1
     modbus_buf[6] = mdns.mdWorkSpace[0x47];//p1     
    }
   flags |= (1 << 1);
   flags &= ~(1 << 0);
   modbus_buf[2] = flags;   
  }
 else if(mdns.mdfl[Conc] >= mdns.mdfl[P1])
  {
   if(flags&0x03 == 0)         //если небыло до этого порога
    {
     mdns.mdui[P1Count]++;
     WriteByteIntoExtEEPROM(0X3E, mdns.mdWorkSpace[0X46]); 
     WriteByteIntoExtEEPROM(0X3F, mdns.mdWorkSpace[0X47]);
     modbus_buf[9] = mdns.mdWorkSpace[0x48];//p2
     modbus_buf[8] = mdns.mdWorkSpace[0x49];//p2     
    }
   flags |= (1 << 0);      
   flags &= ~(1 << 1);
   modbus_buf[2] = flags;   
  }
 else
  {
   flags &= ~(1 << 0);
   flags &= ~(1 << 1);
   modbus_buf[2] = flags;    
  } 
  if(_s_disrec == 0)
   stop_disrec |= 0x0F;
 }


если упрощённо, то функция делает следующее:

Код
void processing_data(void)
{
расчитываем некое значение mdns.mdfl[Conc]
  
 if(проверяем - невышло ли значение сверх всяких норм)
  если да, то орём всем, чем можем и записываем сей факт в журнале

 else if(проверяем - не слишком ли близко подошло значение к "сверх всяких норм")
  {
    верещим пищалкой, зажигаем два светодиода и делаем запись в журнале
  }

 else if(проверяем - не превысило ли значение из ряда "нормальных значений")
  {
    верещим пищалкой, зажигаем один светодиод и делаем запись в журнале
  }

 else
  {
   всё хорошо, продолжаем как ни в чём небыло
  } 

 }


Так вот - вне зависимости от значения во второй и тетий if программа незаходит. Т.е. зачение расчитывается, я его спрашиваю по модбасу, оно соответсвует действительности, но ни один светодиод не загорается и звуковой извещатель всё время молчит. Поэтому я даже не знаю - толи эти if'ы не вошли в готовую программу, либо всё время заходит в последний иф. В живую получить значения выше mdns.mdfl[Pr] у меня нет возможности, поэтому этот if мне не проверить.

В процессе отладки я закомментировал расчёты mdns.mdfl[Conc] просто присваивал этой переменной различные значения. Результат тотже (это в принципе неудивительно, тут ещё можно поверить оптимизатору), но если я этой переменной присваивал значение выше чем mdns.mdfl[Pr], то в первый if программа заходит. Из этого я могу сделать вывод, что оптимизации поддались все остальные if'ы. Можно ли это как-нибудь побороть?


прочие исходные данные : avr-gcc (WinAVR 20080512) 4.3.0; уровень оптимизации "s" (меньше ставить нельзя, больше смысла нет).


--------------------
recaptcha is great
PM MAIL ICQ Skype   Вверх
Mazzi
Дата 4.2.2009, 17:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Правильный
**


Профиль
Группа: Комодератор
Сообщений: 825
Регистрация: 3.4.2003

Репутация: 9
Всего: 21
Если ты будешь задавать в программе литералы, то от оптимимзации не избавишься.(наверное smile)

Сделай так, помести вызов этой функции в цикл, в цикле на каждом проходе меняй переменные влияющие на ветвления и вызывай эту функцию.  А затем проверь что вышло в симуляторе.

Удачи!


--------------------
Мне нужны помощники.
PM MAIL WWW   Вверх
Cr@$h
Дата 4.2.2009, 21:27 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Исследователь
***


Профиль
Группа: Участник Клуба
Сообщений: 1693
Регистрация: 3.4.2005
Где: Санкт-Петербург, Россия

Репутация: нет
Всего: 41
Цитата(UniBomb @  4.2.2009,  17:51 Найти цитируемый пост)
уровень оптимизации "s" (меньше ставить нельзя, больше смысла нет). 

Цитата(Mazzi @  4.2.2009,  18:10 Найти цитируемый пост)
Если ты будешь задавать в программе литералы, то от оптимимзации не избавишься.

Используй -Od, чтобы код компилировался "как он есть" безо всяких оптимизаций. Тогда либо получится, либо хотя бы на неё грешить не будешь.
PM MAIL ICQ   Вверх
UniBomb
Дата 5.2.2009, 08:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок
***
Награды: 1



Профиль
Группа: Завсегдатай
Сообщений: 1754
Регистрация: 24.10.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 97
Цитата(Mazzi @  4.2.2009,  17:10 Найти цитируемый пост)
Сделай так, помести вызов этой функции в цикл, в цикле на каждом проходе меняй переменные влияющие на ветвления и вызывай эту функцию.  А затем проверь что вышло в симуляторе.

Эта функция и так вызывается циклически - по таймеру. В процесе отладки я менял значения mdns.mdfl[Conc] в симуляторе (AVRStudio) всё было хорошо - я попадал в нужные ветвления и нормально выполнялась работа, а зашиваю в мк и фиг.


Цитата(Cr@$h @  4.2.2009,  21:27 Найти цитируемый пост)
Используй -Od, чтобы код компилировался "как он есть" безо всяких оптимизаций.

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


--------------------
recaptcha is great
PM MAIL ICQ Skype   Вверх
UniBomb
Дата 9.2.2009, 14:02 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок
***
Награды: 1



Профиль
Группа: Завсегдатай
Сообщений: 1754
Регистрация: 24.10.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 97
Всем спасибо за внимание, проблема разрешилась. Я немного упростил эту функцию и часть её вынес в другую. Всё заработало, хотя о причинах этого глюка я до сих пор могу только догадываться.


--------------------
recaptcha is great
PM MAIL ICQ Skype   Вверх
Cr@$h
Дата 9.2.2009, 14:11 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Исследователь
***


Профиль
Группа: Участник Клуба
Сообщений: 1693
Регистрация: 3.4.2005
Где: Санкт-Петербург, Россия

Репутация: нет
Всего: 41
Цитата(UniBomb @  5.2.2009,  09:52 Найти цитируемый пост)
Тогда размер пограммы будет превышать память мк и попросту не влезет в него.

Проверял?
Цитата(UniBomb @  9.2.2009,  15:02 Найти цитируемый пост)
Всё заработало, хотя о причинах этого глюка я до сих пор могу только догадываться. 

Если есть возможность, можно сделать профилирование и посмотреть, какие ветки сколько раз использовались..
PM MAIL ICQ   Вверх
OlegIT
Дата 9.2.2009, 15:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 684
Регистрация: 5.4.2006

Репутация: нет
Всего: нет
Обратите внимание на volatile. Сталкивался с такой проблемой, мне помогло.
PM MAIL   Вверх
UniBomb
Дата 9.2.2009, 21:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок
***
Награды: 1



Профиль
Группа: Завсегдатай
Сообщений: 1754
Регистрация: 24.10.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 97
Цитата(Cr@$h @  9.2.2009,  14:11 Найти цитируемый пост)
Проверял?

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


Цитата(Cr@$h @  9.2.2009,  14:11 Найти цитируемый пост)
Если есть возможность, можно сделать профилирование

Чего? о_О


Цитата(OlegIT @  9.2.2009,  15:18 Найти цитируемый пост)
Обратите внимание на volatile.

Обратил...


--------------------
recaptcha is great
PM MAIL ICQ Skype   Вверх
Cr@$h
Дата 9.2.2009, 23:35 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Исследователь
***


Профиль
Группа: Участник Клуба
Сообщений: 1693
Регистрация: 3.4.2005
Где: Санкт-Петербург, Россия

Репутация: нет
Всего: 41
Цитата(UniBomb @  9.2.2009,  22:40 Найти цитируемый пост)
Цитата(Cr@$h @  9.2.2009,  14:11 Найти цитируемый пост)
Если есть возможность, можно сделать профилирование

Чего? о_О

Хотя, раз с размером проблемы...
Тогда всё дело в оптимизациях этого размера. Примудрости из разряда style-of-art.
PM MAIL ICQ   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "Микроконтроллеры (MCU) и микропроцессоры (MPU)"
PILOT ManiaK
UniBomb Mazzi

На данный раздел помимо Правил форума распространяются текже следующие правила:


  • Прежде чем создать тему воспользуйтесь поиском или посмотрите в faq. Возможно на форуме уже есть ответ на ваш или близкий к вашему вопрос.
  • В заголовке темы в квадратных скобках обозначьте используемое семейство микроконтроллера: [avr],[pic],[arm].
  • При создании темы с вопросом указывайте участок кода с ошибкой, версию компилятора, схемы подключения, fuse биты и прочие данные, которые помогут найти правильный ответ. Для форматирования текста программ используйте кнопку код.
  • Новое сообщение должно иметь прямое отношение к тематике этого раздела. Для флуда, просьб выполнить задание, поиска партнёров или исполнителей существуют свои разделы.
  • Если вы заметили несовместимое с правилами сообщение, то можете уведомить об этом модератора раздела нажав кнопку Репорт у соответствующего сообщения.

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, PILOT, ManiaK, UniBomb, Mazzi.
 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Микроконтроллеры (MCU) и микропроцессоры (MPU) | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0798 ]   [ Использовано запросов: 20 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.