Кто любит RISC в жизни, заходим, не стесняемся.
Ответить

Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 09:36:48

C8T6, 72Мгц, внешний кварц, CMSIS без каких либо надстроек, g++

имеется прерывание 4мкс, он же счетчик для событий + необходимо некоторые данные обновлять \ считать

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

a = (((b * 10) + (c / 60)) - 1) * 5; - прерывание будет выполняться 5.184135 мкс

а вот такая конструкция:

x = b * 10;
y = c / 60;
a = x + y - 1;
a *= 5;


не будет влиять на время прерывания, т.е. время прерывания чётко 4.000000мкс

что компилятору там могло в голову ударить ? :shock:

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 09:55:29

V2oD2o писал(а):a = (((b * 10) + (c / 60)) - 1) * 5;
ну а если записать "естественно", без лишних скобок:a = (b*10 + c/60 - 1) * 5;? что будет?

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 10:04:27

Какие типы переменных? Float или Double среди них есть?
Результат первой и второй конструкции идентичен или отличается?
Оптимизация по скорости включена?

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 10:29:55

V2oD2o писал(а):a = (((b * 10) + (c / 60)) - 1) * 5;
ну а если записать "естественно", без лишних скобок:a = (b*10 + c/60 - 1) * 5;? что будет?


Пример условный, но закономерность именно такая, всё считаешь в одной строке - лаг по времени прерывания, разделяешь - все нормально

Добавлено after 2 minutes 18 seconds:
Какие типы переменных? Float или Double среди них есть?
Результат первой и второй конструкции идентичен или отличается?
Оптимизация по скорости включена?


C8T6 не имеет FPU, в первом посте указал что только CMSIS без всяких надстроек т.е. никаких извращений с float на цельночисленном камне, тип данных uint16_t или 32_t - ситуация одинаковая, оптимизации пробовал разных уровней - результат малоотличим

Данные снимал анализатором, в прерывании ->ODR ^= и смотрел на интервалы прерывания

И еще, для понимания - калькуляция там не одна, их с десяток наберется, за пределы результирующей длины слова - не вылезает, по отладчику перепроверено несколько раз

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 10:35:45

ну а если записать "естественно", без лишних скобок:a = (b*10 + c/60 - 1) * 5;? что будет?

А есть варианты? Твоя программа разбора выражений со скобками и без могла выдавать разные результаты?

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

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 10:40:11

ну а если записать "естественно", без лишних скобок:a = (b*10 + c/60 - 1) * 5;? что будет?

А есть варианты? Твоя программа разбора выражений со скобками и без могла выдавать разные результаты?

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


выражение условное, там несколько десятков расчётов завязанные на разные временные события и внешние прерывания, основная масса расчетов ведётся по данным ADC через DMA - 4 канала

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

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

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 11:37:48

тип данных uint16_t или 32_t - ситуация одинаковая

В многострочном варианте промежуточный результат явно приводится к типу присваиваемых переменных, в однострочном - нет. Проверьте, интересу ради, вариант a = (uint_16t)(((uint_16t)(b * 10) + (uint_16t)(c / 60)) - 1) * 5; Кроме того, теоретически, операции "перемножение" и "домножение" имеют разное количество аргументов - и компилятор вполне может решить пользоваться разными инструкциями буде такие в его распоряжении на рассматриваемой платформе. А сгенерированный ASM код наверняка неидентичен - может туда глянуть? ;-)

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:00:32

тип данных uint16_t или 32_t - ситуация одинаковая

В многострочном варианте промежуточный результат явно приводится к типу присваиваемых переменных, в однострочном - нет. Проверьте, интересу ради, вариант a = (uint_16t)(((uint_16t)(b * 10) + (uint_16t)(c / 60)) - 1) * 5; Кроме того, теоретически, операции "перемножение" и "домножение" имеют разное количество аргументов - и компилятор вполне может решить пользоваться разными инструкциями буде такие в его распоряжении на рассматриваемой платформе. А сгенерированный ASM код наверняка неидентичен - может туда глянуть? ;-)


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

Насчет приведения к типам - хорошая идея, попробую вечером

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:03:29

Эту строку.
Код:
a = (((b * 10) + (c / 60)) - 1) * 5;
GCC компилировал так (все переменные uint32_t).
Код:
//      a = (((b * 10) + (c / 60)) - 1) * 5;
080004AC   ldr   r1, [pc, #64]   ; (0x80004f0 <main+116>)
080004B0   add.w   r2, r2, r2, lsl #2
080004B4   umull   r1, r3, r1, r3
080004B8   lsrs   r3, r3, #5
080004BA   add.w   r3, r3, r2, lsl #1
080004BE   add.w   r3, r3, r3, lsl #2
080004C2   subs   r3, #5
Эти строки.
Код:
    x = b * 10;
    y = c / 60;
    a = x + y - 1;
    a *= 5;
скомпилировались в такой код.
Код:
//      x = b * 10;
080004B0   add.w   r2, r2, r2, lsl #2
//      y = c / 60;
080004AC   ldr   r1, [pc, #64]   ; (0x80004f0 <main+116>)
080004B4   umull   r1, r3, r1, r3
080004B8   lsrs   r3, r3, #5
//      a = x + y - 1;
080004BA   add.w   r3, r3, r2, lsl #1
080004BE   subs   r3, #1
//      a *= 5;
080004C0   add.w   r3, r3, r3, lsl #2

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:12:03

Эту строку.
Код:
a = (((b * 10) + (c / 60)) - 1) * 5;
GCC компилировал так (все переменные uint32_t).
Код:
//      a = (((b * 10) + (c / 60)) - 1) * 5;
080004AC   ldr   r1, [pc, #64]   ; (0x80004f0 <main+116>)
080004B0   add.w   r2, r2, r2, lsl #2
080004B4   umull   r1, r3, r1, r3
080004B8   lsrs   r3, r3, #5
080004BA   add.w   r3, r3, r2, lsl #1
080004BE   add.w   r3, r3, r3, lsl #2
080004C2   subs   r3, #5
Эти строки.
Код:
    x = b * 10;
    y = c / 60;
    a = x + y - 1;
    a *= 5;
скомпилировались в такой код.
Код:
//      x = b * 10;
080004B0   add.w   r2, r2, r2, lsl #2
//      y = c / 60;
080004AC   ldr   r1, [pc, #64]   ; (0x80004f0 <main+116>)
080004B4   umull   r1, r3, r1, r3
080004B8   lsrs   r3, r3, #5
//      a = x + y - 1;
080004BA   add.w   r3, r3, r2, lsl #1
080004BE   subs   r3, #1
//      a *= 5;
080004C0   add.w   r3, r3, r3, lsl #2


Уровень оптимизации какой ?
Судя по строке
080004C2 subs r3, #5 - что то пошло не так
Последний раз редактировалось V2oD2o Пн июл 09, 2018 13:13:42, всего редактировалось 1 раз.

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:12:51

-Ofast.
Необязательно для всего проекта. Можно только для нужной функции.

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:14:25

-Ofast.
Необязательно для всего проекта. Можно только для нужной функции.


Обновил пред коммент, откуда там?
080004C2 subs r3, #5

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 13:36:43

откуда там?
080004C2 subs r3, #5

Современные компиляторы очень умные, в данном случае он раскрыл скобки(b * 50 + c / 12 - 5), заменил деление умножением, умножение заменил сдвигами и сложением, более того т.к. из-за целочисленного деления результат упрощенного выражения может быть другим, то было предусмотрено и это. Так что не нужно искать баги у компилятора, ищи баги у себя :)

Re: Наткнулся на какой то баг компилятора?

Пн июл 09, 2018 17:11:17

V2oD2o , покажите полносттю обработчик прерывания таймера.
Особенно интересует сброс флага...

Re: Наткнулся на какой то баг компилятора?

Вт июл 10, 2018 09:20:24

V2oD2o , покажите полносттю обработчик прерывания таймера.
Особенно интересует сброс флага...


Там все в порядке, проект рабочий, после доработок, не сразу, но было замечено увеличение времени отсчета по таймеру примерно х1.8 - х2, т.е. событие по 10мс, выполнялось в 18.5064мс

Добавлено after 7 minutes 45 seconds:
тип данных uint16_t или 32_t - ситуация одинаковая

В многострочном варианте промежуточный результат явно приводится к типу присваиваемых переменных, в однострочном - нет. Проверьте, интересу ради, вариант a = (uint_16t)(((uint_16t)(b * 10) + (uint_16t)(c / 60)) - 1) * 5; Кроме того, теоретически, операции "перемножение" и "домножение" имеют разное количество аргументов - и компилятор вполне может решить пользоваться разными инструкциями буде такие в его распоряжении на рассматриваемой платформе. А сгенерированный ASM код наверняка неидентичен - может туда глянуть? ;-)


"Дело было не в бобине" - и действительно, приведение к типам дало прирост, очень ощутимый, было 18мс вместо 10мс, перевёл все расчеты на u32_t (в 16 не влезу) - стало 13.3мс, добавил -O3 - и увидел заветные 10мс, пробовал нагрузить еще всякими вычислениями, даже гораздо более сложными и считать их каждый тик, а не по событию - проблема решилась, каждый тик ровно 5.000 мкс, спасибо за наводку! 8)

Что интересно -Ох без использования однотипных данных - почти ничего не давало

Re: Наткнулся на какой то баг компилятора?

Вт июл 10, 2018 15:10:05

Особенно интересует сброс флага...

И все же, многие влетали на эти грабли сбрасывая флаг в конце распухшего прерывания...

Re: Наткнулся на какой то баг компилятора?

Вт июл 10, 2018 15:15:59

Особенно интересует сброс флага...

И все же, многие влетали на эти грабли сбрасывая флаг в конце распухшего прерывания...


Да, я не исключение, проходили
Ответить