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

 
 
3 страниц V  < 1 2 3 >  
Reply to this topicStart new topic
> Keil Оптимизатор
scifi
сообщение Jan 16 2012, 20:31
Сообщение #16


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Цитата(aaarrr @ Jan 17 2012, 00:24) *
Странный совет. Правильнее было бы не выключать оптимизацию, дабы по мере написания программы выгребать содержимое ящика.

Подозреваю, что автор - очень начинающий программист на Си. Зачем лишние грабли на дороге?
Go to the top of the page
 
+Quote Post
vlad_new
сообщение Jan 16 2012, 20:32
Сообщение #17


Местный
***

Группа: Участник
Сообщений: 218
Регистрация: 24-06-10
Пользователь №: 58 127



Цитата(aaarrr @ Jan 17 2012, 00:24) *
Подобные симптомы свидетельствуют отнюдь не в пользу версии о правильности программы, а совсем даже наоборот.


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


В KEIL-е без оптимизатора на код смотреть противно. Да и скорость почти 2 раза ниже.
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Jan 16 2012, 20:35
Сообщение #18


Гуру
******

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



Цитата(vlad_new @ Jan 17 2012, 00:30) *
Внешний массив и обязан быть volatile, тем более что он используется в прерываниях.

Нет, не обязан.
Go to the top of the page
 
+Quote Post
scifi
сообщение Jan 16 2012, 20:36
Сообщение #19


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Цитата(vlad_new @ Jan 17 2012, 00:30) *
Естественно. Внешний массив и обязан быть volatile, тем более что он используется в прерываниях. Ну так я и спрашивал как бы
сделать так, чтоб в стандартные библиотеки передовать тип volatile без явного преобразования типа повсеместно.
Может через какой нибудь typedef переопределить типы параметров в библиотеке. А может еще какие то другие есть решения.

Можно тупо объявить указатель на этот массив и использовать его в вызовах библиотечных функций:
Код
char* const nv_Buf_RS485 = (char*)Buf_RS485;

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

Цитата(vlad_new @ Jan 17 2012, 00:32) *
В KEIL-е без оптимизатора на код смотреть противно. Да и скорость почти 2 раза ниже.

Лучше не начинайте. А то я начну рассказывать, какие эмоции вызывает Ваш код :-)
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Jan 16 2012, 20:37
Сообщение #20


Гуру
******

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



Цитата(scifi @ Jan 17 2012, 00:31) *
Зачем лишние грабли на дороге?

Чтобы научиться по этой дороге ходить, разве нет?
Go to the top of the page
 
+Quote Post
vlad_new
сообщение Jan 16 2012, 20:51
Сообщение #21


Местный
***

Группа: Участник
Сообщений: 218
Регистрация: 24-06-10
Пользователь №: 58 127



Цитата(scifi @ Jan 17 2012, 00:36) *
Можно тупо объявить указатель на этот массив и использовать его в вызовах библиотечных функций:
Код
char* const nv_Buf_RS485 = (char*)Buf_RS485;

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


Лучше не начинайте. А то я начну рассказывать, какие эмоции вызывает Ваш код :-)

Думал я про указатель. Иногда так и делаю. Код нормальный с точки зрения 8 битных процессоров, где указатель является очень
сложной конструкцией и работает через библиотеки жутко медленно. ARM-наоборот. Ему только указатели и подавай.
Да я собственно и не претендовал на программиста. На перелопачивание СРМа и MS-DOSа меня в свое время хватало, а Андроид писать и не собираюсь.
Но в любом случае, как я Вас понял, пороблемма снимается изменением стиля работы с буферами. Типа заводим буфер и сразу на него указатель. А про
индексы на всегда збываем. sm.gif
PS: Подредактировано.


Сообщение отредактировал vlad_new - Jan 16 2012, 21:09
Go to the top of the page
 
+Quote Post
scifi
сообщение Jan 16 2012, 21:05
Сообщение #22


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Да, забыл. Можно просто макрос. Ну и я переврал немного с указателем: не нужно его делать const, иначе он ничем не лучше volatile - требует приведения типа.
Go to the top of the page
 
+Quote Post
vlad_new
сообщение Jan 16 2012, 21:08
Сообщение #23


Местный
***

Группа: Участник
Сообщений: 218
Регистрация: 24-06-10
Пользователь №: 58 127



Цитата(scifi @ Jan 17 2012, 01:05) *
Да, забыл. Можно просто макрос. Ну и я переврал немного с указателем: не нужно его делать const, иначе он ничем не лучше volatile - требует приведения типа.

Вот только хотел спросить про const sm.gif Я подредактировал предыдущее сообщение.
Сенкю.
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Jan 16 2012, 21:23
Сообщение #24


Гуру
******

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



Цитата(vlad_new @ Jan 17 2012, 00:51) *
Но в любом случае, как я Вас понял, пороблемма снимается изменением стиля работы с буферами. Типа заводим буфер и сразу на него указатель. А про
индексы на всегда збываем. sm.gif

Позволите позанудствовать немного?

Если преобразование типов действительно необходимо, то лучше выполнить его в момент вызова, а не заводить левый указатель.
Если таких преобразований требуются сотни, значит программу надо переписать.
С указателем можно тоже работать с использованием индекса.
Go to the top of the page
 
+Quote Post
vlad_new
сообщение Jan 16 2012, 21:35
Сообщение #25


Местный
***

Группа: Участник
Сообщений: 218
Регистрация: 24-06-10
Пользователь №: 58 127



Цитата(aaarrr @ Jan 17 2012, 01:23) *
Позволите позанудствовать немного?

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

Хороший совет тем, кто перелез с 8 битников на ARMы, а таких сейчас очень много.
К стате почему эту строчку оптемизатор отбрасывает - понятно. Поскольку Buf_RS485[0] нигде далее не используется, она, типа, и не нужна! А то что
это нужно прерыванию компелятор в упор не видит. А зря. Вот и приходится либо где нибудь влепить холостую дурь типа k=Buf_RS485[0];
либо терять скорость на куче volatile.
Но вопрос то стоял не почему оптимизатор в Keil-е дурной sm.gif , а как в библиотеке подмену на volatile сделать. Хотя может это и не возможно, поскольку
библиотека уже оптимизирована на регистровые операции и заставить библиотечные ф-ции сбрасывать регистровые переменные в ОЗУ вряд ли возможно.
А может я и ошибаюсь. Собственно я это и хотел выяснить.

Сообщение отредактировал vlad_new - Jan 16 2012, 23:38
Go to the top of the page
 
+Quote Post
scifi
сообщение Jan 17 2012, 06:16
Сообщение #26


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Опять всё по кругу...

Цитата(vlad_new @ Jan 17 2012, 01:35) *
К стате почему эту строчку оптемизатор отбрасывает - понятно. Поскольку Buf_RS485[0] нигде далее не используется, она, типа, и не нужна!

Абсолютно верно.

Цитата(vlad_new @ Jan 17 2012, 01:35) *
А то что это нужно прерыванию компелятор в упор не видит. А зря.

Компилятор вообще много чего не видит и не должен видеть. Вы как будто пытаетесь ему приписать роль этакого телепата-волшебника, угадывающего намерения программиста. А программист на что?

Цитата(vlad_new @ Jan 17 2012, 01:35) *
Вот и приходится либо где нибудь влепить холостую дурь типа k=Buf_RS485[0];
либо терять скорость на куче volatile.

Лепить холостую дурь не надо.
А вот одно из применений volatile - для переменных, используемых как в основном потоке, так и в прерывании. Действительно, каждый доступ к volatile потенциально замедляет программу. Поэтому число таких доступов уменьшают до минимального необходимого уровня. Есть как минимум несколько способов это сделать.

Цитата(vlad_new @ Jan 17 2012, 01:35) *
Но вопрос то стоял не почему оптимизатор в Keil-е дурной sm.gif , а как в библиотеке подмену на volatile сделать. Хотя может это и не возможно, поскольку
библиотека уже оптимизирована на регистровые операции и заставить библиотечные ф-ции сбрасывать регистровые переменные в ОЗУ вряд ли возможно.

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

Цитата(vlad_new @ Jan 17 2012, 01:35) *
А может я и ошибаюсь. Собственно я это и хотел выяснить.

Вы ошибаетесь :-)
Налицо все признаки ужасно структурированной программы. Отсюда и лезут все эти косяки. По неопытности бывает, конечно. У меня в начале тоже программы были не фонтан.
Go to the top of the page
 
+Quote Post
_Ivana
сообщение Jan 17 2012, 17:07
Сообщение #27


Местный
***

Группа: Свой
Сообщений: 352
Регистрация: 13-08-11
Из: Воронеж
Пользователь №: 66 710



Продолжу от лица неопытных новичков rolleyes.gif
Можно было бы конечно продублировать все библиотечные функции своими аналогами, внутри которых заводятся локальные переменные нужных типов, им присваиваются значения параметров и с ними вызываются библиотечные функции. Но мне кажется, это несколько коряво, хотя как лучше - не знаю.
Простейший пример из практики - преобразование числа в строку. Мне было проще написать эту функцию самому, чем копаться в библиотеках. Она принимает аргумент типа int. А мне надо вызывать её и с int и с char. Два решения лежат на поверхности:
1 - написать дубль этой же функции, но принимающий char
2 - заводить лишнюю переменную int куда присваивать нужный char, и с ним вызывать функцию
Оба решения мне не нравятся. Но как сделать красивее - не придумал. Использование указателя не только не помогло, но и таит ошибку при исполнении.
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Jan 17 2012, 17:14
Сообщение #28


Гуру
******

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



Цитата(_Ivana @ Jan 17 2012, 21:07) *
Два решения лежат на поверхности

А почему бы не передавать ей char без лишних телодвижений?
Go to the top of the page
 
+Quote Post
_Ivana
сообщение Jan 17 2012, 17:22
Сообщение #29


Местный
***

Группа: Свой
Сообщений: 352
Регистрация: 13-08-11
Из: Воронеж
Пользователь №: 66 710



Хороший вопрос! sm.gif
Я сначала написал, а потом начал что-то вспоминать про "неявное преобразование типов" и т.п. Будете смеяться, но не помню, почему я был вынужден так изголяться... Или там передавался наоборот указатель на int...

ЗЫ щас попробую найти код и посмотреть sm.gif
Точно, принимаю указатель sm.gif Чтобы переменная изменялась а не передалась по значению и все.

Код
//-----------------------------------------------------------------------------------------------

char count_of_order(unsigned int *n, unsigned int order)
{
    char res = 0;

    while (*n >= order)
    {
        *n = *n - order;
        res++;
    }

    return res;
}
//-----------------------------------------------------------------------------------------------


char *int_to_string(char *s, unsigned int n)
{    
    char *begin = s;
    char digit;

    if((digit = count_of_order(&n, 10000))    || s!=begin) *s++ = '0' + digit;
    if((digit = count_of_order(&n, 1000))    || s!=begin) *s++ = '0' + digit;
    if((digit = count_of_order(&n, 100))     || s!=begin) *s++ = '0' + digit;
    if((digit = count_of_order(&n, 10))     || s!=begin) *s++ = '0' + digit;

    *s++ = '0' + n;

    return s;
}
//-----------------------------------------------------------------------------------------------


а в другом месте нужен вызов с указателем на char

Код
//-----------------------------------------------------------------------------------------------


char *GPS_end_of_string(char *begin, char *end)
{    
    char *d;
    char chek_summ = 0;
    unsigned int int_chek_summ;

    for(d=begin+1; d<end; d++) chek_summ = chek_summ^*d;
    int_chek_summ = chek_summ;

    *d++ = GPS_char_multiply;
    *d++ = '0' + count_of_order(&int_chek_summ, 16);
    *d++ = '0' + int_chek_summ;
    *d++ = GPS_char_enter;
    *d++ = GPS_char_return;
    *d++ = 0;

    return d;
}
//-----------------------------------------------------------------------------------------------


ЗЫ вышеприведенный код содержит ошибку - это не финальная редакция, я её потом исправил - тут неправильно пишутся буквы 16-ричной строки числа, поскольку A, B и т.д. идут не сразу после "9" в таблице символов sm.gif Но на принципиальном вопросе это не отражается, а его напомню кратко - нам надо передавать не числовое значение переменной в функцию, а указатель на неё, чтобы функция в процессе своего выполнения изменяла этот параметр. А при передаче указателя на char в указатель на int он обрастет ещё соседним байтом и будет ошибка.

Сообщение отредактировал _Ivana - Jan 17 2012, 17:43
Go to the top of the page
 
+Quote Post
aaarrr
сообщение Jan 17 2012, 18:13
Сообщение #30


Гуру
******

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



Зачем, нет ЗАЧЕМ делить на 16 последовательным вычитанием? Можно просто написать n >> 4.

С указателями - да, никакой автоматики не предусмотрено.
Go to the top of the page
 
+Quote Post

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

 


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


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