🚀 go-pugleaf

Your translations is what I had, but it always failed to compile, like this:

void __fastcall TForm1::Button1Click(TObject *Sender)
{
TRegisters Registers;

asm
 {
  PUSH    EBX
  PUSH    EDI
  MOV     EDI, Registers
  XOR     EBX, EBX
  XOR     ECX, ECX
  XOR     EDX, EDX
  DB 0x0F, 0xA2
  MOV     [EDI], EAX
  MOV     [EDI+4], EBX
  MOV     [EDI+8], ECX
  MOV     [EDI+12], EDX
  POP     EDI
  POP     EBX
 }
}

When I moved the code to another function (like you did), it compiled.

What's the difference between the above and this:

void __fastcall TForm1::Button1Click(TObject *Sender)
{
TRegisters Registers;
GetCPUID( 0, Registers );
}

void __fastcall TForm1::GetCPUID(DWORD Param, TRegisters &Registers)
{
asm
 {
  PUSH    EBX
  PUSH    EDI
  MOV     EDI, Registers
  XOR     EBX, EBX
  XOR     ECX, ECX
  XOR     EDX, EDX
  DB 0x0F, 0xA2
  MOV     [EDI], EAX
  MOV     [EDI+4], EBX
  MOV     [EDI+8], ECX
  MOV     [EDI+12], EDX
  POP     EDI
  POP     EBX
 }
}

and why do I need to pass the DWORD Param (it would not compile without it).

-- 
Malcolm Smith



"Jonathan Benedicto" <invalid@nobody.com> wrote in message 
news:46a0dcbb$1@newsgroups.borland.com...
> Malcolm Smith wrote:
>> I'm trying to convert this PAS code into a C++ unit but I am not having 
>> any luck with the ASM code - it won't compile as is (although the PAS 
>> file does).
>>
>> Can someone please help with the ASM translation.  I know I could just 
>> use the .PAS file but I want to learn something along the way.
>
> I'm no expert at ASM, but I'd guess that this would be the C++ 
> translation:
>
> struct TRegisters
> {
> DWORD EAX;
> DWORD EBX;
> DWORD ECX;
> DWORD EDX;
> };
>
> void __fastcall GetCPUID( DWORD Param, TRegisters &Registers )
> {
> asm
> {
>  PUSH    EBX                         /* save affected registers */
>  PUSH    EDI
>  MOV     EDI, Registers
>  XOR     EBX, EBX                    /* clear EBX register */
>  XOR     ECX, ECX                    /* clear ECX register */
>  XOR     EDX, EDX                    /* clear EDX register */
>  DB 0x0F, 0xA2                       /* CPUID opcode */
>  MOV     [EDI], EAX       /* save EAX register */
>  MOV     [EDI+4], EBX       /* save EBX register */
>  MOV     [EDI+8], ECX       /* save ECX register */
>  MOV     [EDI+12], EDX      /* save EDX register */
>  POP     EDI                         /* restore registers */
>  POP     EBX
> }
> }
>
> Worked for me at any rate.
>
> Jon