I think I find a bug

[1989lzhh]

Code: Select all

#include <stdio.h>
#include <nds.h>
#include <malloc.h>

u16 m[1000]={0};

void WriteRAM(u32 address,const u8* buffer,u32 size)
{       u32 i=0;
	u16* addr=m+address;  
	u16* pData=(u16*)buffer;   
	for(i=0;i<size;i+=2)   
	{      
		addr[i>>1]=pData[i>>1];
	}		
}
void ReadRAM(u32 address,u8* buffer,u32 size)
{
	u32 i=0;
	u16* addr=m+address;  
	u16* pData=(u16*)buffer;
	for(i=0;i<size;i+=2)
	{
		pData[i>>1]=addr[i>>1];   
//Here!The ReadRAM function will work fine if I add i++,i--
                }
//Here!The ReadRAM function will work fine if I put a iprintf() blew.
//iprintf("");
}

//---------------------------------------------------------------------------------
int main(void) {
//---------------------------------------------------------------------------------
    consoleDemoInit();
    int in[3]={10,1100000,20};
    int out[3]={0};
    WriteRAM(0,(u8*)in,sizeof(int)*3);
    ReadRAM(0,(u8*)out,sizeof(int)*3);
    iprintf("%d %d %d\n",out[0],out[1],out[2]);
    while(1);
   
}

I compiled it with devkitARM r31 & libnds1.4.6. In the ReadRAM function code ,If I didn't put a iprintf() at the end,the function didn't work and iprintf("%d %d %d\n",out[0],out[1],out[2]); will get 0 0 0.

[WinterMute]

You're passing a pointer to the out array into the ReadRAM function and not marking the memory it points to as volatile. The compiler then assumes you're not changing the memory. Using volatile int out[3]; makes everything work as expected.