diff options
| author | Torok Edwin <edwintorok@gmail.com> | 2009-04-25 10:25:12 +0000 |
|---|---|---|
| committer | Torok Edwin <edwintorok@gmail.com> | 2009-04-25 10:25:12 +0000 |
| commit | c8afb09a3bec5070ce4ebf0e217d833c3b48443b (patch) | |
| tree | 64c159118198e9a05b733d367f72db28311428da /llvm/lib/System/Path.cpp | |
| parent | 362acf8a561082eed652a246fb52ced42235ee3d (diff) | |
| download | bcm5719-llvm-c8afb09a3bec5070ce4ebf0e217d833c3b48443b.tar.gz bcm5719-llvm-c8afb09a3bec5070ce4ebf0e217d833c3b48443b.zip | |
Fix g++-4.4.0 warning, it was causing llvm-nm to fail on wrapped BC files:
Path.cpp:59: warning: case label value exceeds maximum value for type
magic[0] is a (signed) char, but some case values are unsigned (e.g. 0xde).
When magic[0] was 0xde, the switch has taken the default branch instead of case
0xde branch.
Apparently this was the behaviour with older versions of gcc too, but not with g++.
Now g++-4.4 behaves as gcc, and ignores unsigned case values out of range signed
range.
llvm-svn: 70038
Diffstat (limited to 'llvm/lib/System/Path.cpp')
| -rw-r--r-- | llvm/lib/System/Path.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/System/Path.cpp b/llvm/lib/System/Path.cpp index 4c965db7223..72bd7ad6f04 100644 --- a/llvm/lib/System/Path.cpp +++ b/llvm/lib/System/Path.cpp @@ -55,7 +55,7 @@ LLVMFileType sys::IdentifyFileType(const char *magic, unsigned length) { assert(magic && "Invalid magic number string"); assert(length >=4 && "Invalid magic number length"); - switch (magic[0]) { + switch ((unsigned char)magic[0]) { case 0xDE: // 0x0B17C0DE = BC wraper if (magic[1] == (char)0xC0 && magic[2] == (char)0x17 && magic[3] == (char)0x0B) |
