FIX: /G3 Optimization Generates Bad Code for Struct Pointers

ID: Q116137

1.00 WINDOWS kbprg kbfixlist kbbuglist

The information in this article applies to:

SYMPTOMS

The /G3 option, used to generate 386 instructions, may generate incorrect instructions when attempting to access fields within a structure.

RESOLUTION

There are two workarounds to this problem:

STATUS

Microsoft has confirmed this to be a problem with the C/C++ compiler, version 8.0. The problem was corrected with C/C++, version 8.0c, supplied with Microsoft Visual C++ for Windows, version 1.5. We are researching this problem and will post new information in the Microsoft Knowledge Base as soon as it becomes available.

MORE INFORMATION

The sample code below can be used to illustrate this problem. If you use the /Fc option to generate mixed-source and assembly code, the line commented below generates the following code:

   ;|***    t2 = (lpDisp->right - lpDisp->left);
   ; Line 22
     *** 000027  c4 5e 0c         les bx,DWORD PTR [bp+12]          ;lpDisp
     *** 00002a  66 26 8b 47 0c   mov eax,DWORD PTR es:[bx+12]
     *** 00002f  66 26 2b 47 04   sub eax,DWORD PTR es:[bx+4]
     *** 000034  89 46 f8         mov WORD PTR [bp-8],ax        ;t2
     *** 000037  89 56 fa         mov WORD PTR [bp-6],dx

NOTE: The first three lines of assembly code correctly perform the arithmetic operations, but the last two lines do not use the extended registers "EAX" and "EDX", but "AX" and "DX". Compiling using the /G2 switch instead of the /G3 switch generates correct register use.

Sample Code

/* Compile options needed: /c /G3
*/ 

   #include <windows.h>

   struct displayinfo
   {
    long leftlim, left, rightlim, right;
   };

   void HorizScrollProc(HWND hWnd, WORD wParam, long lParam,
                        struct displayinfo far *lpDisp)
   {
    long t1,t2;

    switch (wParam)
    {
     case SB_LINEUP:
       t1 = (lpDisp->left + lpDisp->right) / 2;
       t2 = (lpDisp->right - lpDisp->left);        /* bad code here */ 
       if (lpDisp->leftlim + t2 / 8 > lpDisp->left)
       {
        lpDisp->left = lpDisp->leftlim;
        lpDisp->right = lpDisp->leftlim + t2;
       }
       else
       {
        lpDisp->left -= t2 / 8;
        lpDisp->right -= t2 / 8;
       }
       break ;

     default:
       return;
       break;
    }
   }

Additional reference words: 1.00 8.00 KBCategory: kbprg kbfixlist kbbuglist KBSubcategory: CLIss
Keywords          : kbCompiler kbbuglist kbfixlist
Version           : 1.00
Platform          : WINDOWS
Solution Type     : kbfix

Last Reviewed: September 21, 1997