Кривая оптимизация XC16

[worldmasters]

Здравствуйте.
Есть чип PIC24FV32KA302 и компилятор XC16.

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

Но вот почему то вышел какой то затык.

вот функция задержки по времени. Сам счетчик времени обновляется по таймеру каждые 10мс.

volatile unsigned int delta = 0;

void BlockDelay(volatile unsigned int msec) {
    ScreenRefreshTime = ms_counter;
    while (1) {
        delta = ms_counter - ScreenRefreshTime;
        //delta = (EllapsedTime(ScreenRefreshTime));
        if (delta ›= msec) {
            break;
        }
        Nop();
    }
}

В отладчике вижу что переменные имеют адекватные значения но delta всегда получается 0.
Уже добавил volatile но оптимизатор все равно режет как то странно.

Кто нибудь знает как можно договорится с оптимизатором??

[akegor]

ScreenRefreshTime = ms_counter;
...
delta = ms_counter - ScreenRefreshTime;

Ну. Ноль. Что тут странного?
Возможно, что-то с областями видимости. Попробуйте обойти этот момент.

(volatile unsigned int msec)

Какой смысл в таком объявлении?

[worldmasters]

Возможно, что-то с областями видимости. Попробуйте обойти этот момент.

Как обойти то?? объявления перенести?

Какой смысл в таком объявлении?

Это уже в порядке бреда. Где то прочитал что такие переменные оптимизатор не трогает.

Ноль. Что тут странного?

750-480 = 0

Действительно чего тут странного.

[akegor]

750-480 = 0

Ну да, только это если нет косяка с "областью видимости".
Не видно объявления ms_counter.

Как обойти то??

Как вариант, сделайте

ScreenRefreshTime = ms_counter;

через промежуточную глобальную переменную.

[AJScorp]

как можно договорится с оптимизатором

Никаких кривых оптимизаций, всё по правилам.

ms_counter в цикле не изменяется, поэтому компилятор вправе оптимизировать постоянное чтение этой переменной. Отладчик же читает реальное значение. Чтобы показать компилятору, что переменная может изменяться извне служит директива volatile.
Остальные "volatile" в приведённом куске кода не нужны.

[worldmasters]

AJScorp
Заработало. Спасибо ))

[akegor]

Заработало

worldmasters, при параноидальной валотилизации незаволотилить ms_counter...
А ведь намякивал

Не видно объявления ms_counter.

[worldmasters]

worldmasters, при параноидальной валотилизации незаволотилить ms_counter...

ну бывает, че.. )))