Есть глобальная структура с 4 переменными которые используются во функции MiDI
По адресу 01F4.

Есть функция:
void MiDI(void){
while(1){
здесь код.
ret //выход
};
};

Данная функция вызывается из другого места по указателю на нее.
Выполнив нужные действия выходим из нее, потом возвращаемся обратно, при минимальной оптимизации все нормально вот асм:
4: void MiDI (void){
+000005EF: C020 RJMP PC+0x0021 Relative jump

Но стоит повысить оптимизацию так вот получается вот что асм
4: void MiDI (void){
+00000444: 93BA ST -Y,R27 Store indirect and predecrement
+00000445: 93AA ST -Y,R26 Store indirect and predecrement
+00000446: EFA4 LDI R26,0xF4 Load immediate
+00000447: E0B1 LDI R27,0x01 Load immediate
+00000448: C020 RJMP PC+0x0021 Relative jump

СОХРАНЯЕТ В СТЕКЕ ДАННЫХ регистры R27, R26!
Он думает цикл бесконечный а, я выхожу из него ! по ret (макрос), и происходит потеря данных, ведь я не восстанавливаю из стека данных.
Как компилятор отучить делать вот так
+00000444: 93BA ST -Y,R27 Store indirect and predecrement
+00000445: 93AA ST -Y,R26 Store indirect and predecrement

Попробуйте выходить по break, а ret поставить после бесконечного цикла.
Цикл с выходом из середины (Википедия). Или см. K&R раздел 3.7. Оператор BREAK.

Решил проблему в лоб, спрятал работу со структурой в функцию processingDI();
Теперь функция с бесконечным циклом MiDI состоит только из переключателя switch и функций. Данных нет. :
void MiDI(void){ //
while(1){
processingDI();
switch(GetExeKey()){ //здесь считываем функцией переменную из структуры, которую //обрабатывает processingDI();
case DI_UP_FAN:{
OutFan(0);
Delay(2, MiDIPr);
UpDownFan(UP);
}
И т.д.

Асм при старте void MiDI(void)
4: void MiDI(void){
+0000045D: C020 RJMP PC+0x0021 Relative jump – переход на processingDI();
Теперь при максимальной оптимизации по скорости все нормально работает



Вот асм функции processingDI();
180: void processingDI(void){
+000001BB: 93BA ST -Y,R27 Store indirect and predecrement
+000001BC: 93AA ST -Y,R26 Store indirect and predecrement
181: key.currentKey = ReadDI();
Здесь еще код для работы сос структурой
: key.previousKey = key.currentKey;
+000001CD: 930C ST X,R16 Store indirect
190: };
+000001CE: 91A9 LD R26,Y+ Load indirect and postincrement
+000001CF: 91B9 LD R27,Y+ Load indirect and postincrement
+000001D0: 9508 RET Subroutine return

Каков смысл городить ret (макрос), если стандартного return(); вполне достаточно для корректного выхода из любого цикла, да и из функции заодно?

Error[Pe118]: a void function may not return a value E:\Documens\Alex\Ìîè äîêóìåíòû\MUKS-01\MUKS01v02\task.c 10
это на return();

Все верно ошибку компилятор выдает. "Пустая" функция не может возвращать значение.
И все же стандартизованный способ прерывания цикла это оператор break. В крайнем случае goto на метку вне цикла.

А что оно говорит если написать так return; // без скобочек ?


return это вполне валидный способ прекращения цикла,
а вот goto - не всегда
при использовании goto можно легко получать переполнение стека,
если его использовать необдуманно

Легко? Cтрашилка. Никакое необдуманоое и даже преднамеренное "плохое" использование goto не должно приводить к переполнению стека или каким либо другим деструктивным последствиям при сколь-нибудь пригодном для работы компиляторе языка высокого уровня. Потеря оптимальности кода может в некоторых "необдуманных" случаях действительно заметной.

goto не возвращает стек в исходное состояние, то есть если переменная была заведена
на стеке то стек будет расходоваться зазря до выхода из функции
ну и в принципе можно нарисовать такую ситуацию когда отведенного стека
просто не хватит
наверное на AVR это не очень актуальная проблемма, но например на
PC легко таким образом организовать переполнение

А return, типа, возвращает, поскольку из функции выходит .
Весь обьем стека под локальные переменные выделяется сразу и соответственно освобождается сразу а не по кусочкам. Другого поведения для компиляторов с языков высокого уровня не наблюдал ни разу. В поминаемом далее PC, вообще комады enter, leave для организации единого стекового фрейма заведены..

Легко, только goto к этому ни сном ни духом.

Мало памяти - мало проблем

Каким таким "таким образом"???

Дело в том, что это вас интересовать не должно. Компилятор (ИАР особенно) - он грамотный, он посмотрит, как все выполняется, и переменную уберет (или не уберет) из стека в нужный момент. Обычно этот момент - всегда выход из функции, т.к. постоянно дергать стек вперед/назад довольно накладно. Так что пользуйте goto, пользуйте return, break, все будет ок, а вот ассемблеровкий ret поперк - это неправильно в любом случае.

И вообще, правильно написаная программа должна компилироваться с макс. оптимизацией и работать. Если такого не происходит, ищите ошибки у себя (обычное дело - volatile забыли, вот как у вас - ret поперек всадили - это даже не ошибка, это... Ну не хочу плохие слова говорить )

Приведите весь код полностью, тогда все ясно будет.

конечно, return; всегда востанавливает указатель стека

Хорошо, сколько по вашему должен расходовать стека данных
следующий код на хорошем компиляторе ?

12 или 22 байта ?
а если будут множественное goto причем как вперед так и назад ?


много регистров меньше проблем

Если без goto - иар постелит 12 байт, если будут goto - 22. Не волнуйтесь за компилятор.

Собственно Rst7 уже ответил. Позволю добавить, что даже при определении переменной внутри оператора место на стеке под нее будет выделено при входе в функцию (оптимизатор может, конечно, затолкнуть переменную и в регистры, но мы сейчас не о том).
Без оптимизации на стеке будет по-любому выделено 22 байта, а уж что оптимизатор сможет наоптимизировать... Это к оптимизатору