diff options
| author | Yotam Barnoy | 2010-08-31 15:24:08 +0000 |
|---|---|---|
| committer | Yotam Barnoy | 2010-08-31 15:24:08 +0000 |
| commit | b5a17ca628e984fa13190c64fb5caef6108369ca (patch) | |
| tree | 1a050aa7a615667131d7e05fcd167858879f99a1 /backends/platform/psp/Makefile | |
| parent | c0b119feda86ea4bc3246868e59e583354656d70 (diff) | |
| download | scummvm-rg350-b5a17ca628e984fa13190c64fb5caef6108369ca.tar.gz scummvm-rg350-b5a17ca628e984fa13190c64fb5caef6108369ca.tar.bz2 scummvm-rg350-b5a17ca628e984fa13190c64fb5caef6108369ca.zip | |
PSP: fix BS2 crash by disabling strict aliasing
The problem that caused the crash in BS2 had to do with GCC making improper assumptions about our code. Specifically, the alignment trick we use for READ_UINT32/16 allows some targets(e.g. MIPS) to generate better code, but it also goes against ANSI C aliasing rules, which prohibit the sharing of addresses between a struct and another variable value. Using -fno-strict-aliasing tells GCC not to assume strict ANSI C aliasing and also unfortunately prevents it from making some good optimizations.
This change is probably needed for other platforms as well -- the crash on the PSP was simply a very rare coincidence.
svn-id: r52473
Diffstat (limited to 'backends/platform/psp/Makefile')
| -rw-r--r-- | backends/platform/psp/Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/backends/platform/psp/Makefile b/backends/platform/psp/Makefile index 617ef7c8cc..249d99c04b 100644 --- a/backends/platform/psp/Makefile +++ b/backends/platform/psp/Makefile @@ -66,7 +66,7 @@ endif # Variables for common Scummvm makefile CXX = psp-g++ -CXXFLAGS = -O3 -Wall -Wno-multichar -fno-exceptions -fno-rtti +CXXFLAGS = -O3 -Wall -Wno-multichar -fno-exceptions -fno-rtti -fno-strict-aliasing DEFINES = -D__PSP__ -DNONSTANDARD_PORT -DDISABLE_TEXT_CONSOLE -DDISABLE_COMMAND_LINE -DUSE_ZLIB -DDISABLE_DOSBOX_OPL -DUSE_RGB_COLOR LDFLAGS := |