В чем заключаются ошибки и как их исправить?

Question

[Андрей Фомин]

Есть такой код на языке С++ с ассемблерной вставкой

#include <iostream>
using namespace std;

int main()
{
    char s1[] = "Hola queridos amigos";
    char tmp;
    asm 
    (  
    "push edi"  
    "push esi"
    "mov edx,lenS"  
    "or edx,edx"  
    "jz _end"   
    "lea eax,s1"  
    "mov esi,eax"  
    "add eax,edx"  
    "dec eax"  
    "mov edi,eax"
        
    "_next:"
    "mov al, [esi]"  
    "mov ah, [edi]"
    "mov [edi], al"
    "mov [esi], ah"
    "inc esi"
    "dec edi"
        
    "cmp edi,esi"
    "jz _end"
    "mov eax,edi"
    "inc eax"
    "cmp eax,esi"
    "jz _end"
    "jmp _next"
        
    "_end:"
    "pop esi"
    "pop edi"
    );
    std::cout<<s1<<std::endl;
    return 0;
}

При его компиляции вылезло следующее:

Error: junk `esimov edx' after expression
Error: too many memory references for `push'

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

Comments

[Евгений Шатунов]

Андрей Фомин , а зачем лог сборки размещать в скриншоте? Это запрещено правилами в П3.8.

[Андрей Фомин]

Евгений Шатунов, Так а что по поводу проблемы скажете, как ее можно решить?

[Евгений Шатунов]

Андрей Фомин, насколько я вижу, ты взял оттранслированный код C++, протащил его через dasm и решил таким образом сделать вставку ассемблера.
Задание тебе не зачтут. Ассемблер пишется иначе.

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

mov edx,lenS - тут непонятно что после запятой.
Стек после завершения функции не выравнен, т.е. число pop и push не совпадает.
Об этом тебе сборщик и пишет в сообщениях.

[Андрей Фомин]

Евгений Шатунов, Я понял, буду тогда редактировать код и разбираться лучше

[Андрей Фомин]

Евгений Шатунов, А как количество push и pop не совпадает, два push и два pop в функции end. к которой мы в любом случае придем, можете поподробнее объяснить этот момент?

Answer 1

[jcmvbkbc]

А в чем заключаются эти ошибки, я не могу понять

Когда ты на С++ пишешь строки подряд, вот так:

"push edi"  
"push esi"
"mov edx,lenS"
...

они склеиваются в одну строку. Когда ассемблер видит команду "push edipush esimov edx, lenS..." он резонно недоумевает.

как их исправить

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

Comments

[Андрей Фомин]

jcmvbkbc, вот возник вопрос: для чего в Ассемблере используются эти символы: $ и %?

[jcmvbkbc]

Андрей Фомин, $ ставят в x86 at&t ассемблере перед константами. % -- перед именами регистров.
$ может показаться излишеством, но он позволяет отличить адрес памяти от содержимого памяти по этому адресу:
mov x, %eax -- загрузка в eax значения из памяти по адресу x
mov $x, %eax -- загрузка в eax самого адреса x.

В интеловском ассемблере для этого используют квадратные скобки и слово offset.

[pfemidi]

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

jcmvbkbc, а если у него не gcc, а, скажем, Visual Studio? Майкрософтовские компиляторы разве понимают ассемблерные вставки в стиле gcc?

[jcmvbkbc]

а если у него не gcc, а, скажем, Visual Studio?

pfemidi, тогда конструкция asm ("... генерирует синтаксическую ошибку компилятора.

[pfemidi]

jcmvbkbc, у него явно не gcc, даже синтаксис ассемблера правильный, интеловский (слева приёмник, справа источник), а не через с ног на голову как в DEC.

[jcmvbkbc]

у него явно не gcc

pfemidi, вы можете говорить с "ним" напрямую, без моего участия.

[pfemidi]

jcmvbkbc, Ok. Но мои комментарии относятся не его словам.

[jcmvbkbc]

pfemidi, мне ваши комментарии в этом треде бесполезны в силу своей очевидности.