6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-10-28 17:23:27 +00:00
Code Issues Projects Releases Wiki Activity

fix an obvious bug found by clang++ and collapse a redundant if.

Browse Source 
Here's the diagnostic from clang:

/Volumes/Data/dgregor/Projects/llvm/lib/Target/CppBackend/CPPBackend.cpp:989:23: warning: 'gv' is always NULL in this context
        printConstant(gv);
                      ^
1 diagnostic generated.



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@91318 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner
2009-12-14 19:34:32 +00:00
parent a4f8ecd85d
commit cdfb302876
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lib/Target/CppBackend/CPPBackend.cpp
View File

@@ -976,21 +976,20 @@ namespace {
nl(Out); nl(Out);
printType(GV->getType()); printType(GV->getType());
if (GV->hasInitializer()) { if (GV->hasInitializer()) {
Constant* Init = GV->getInitializer(); Constant *Init = GV->getInitializer();
printType(Init->getType()); printType(Init->getType());
if (Function* F = dyn_cast<Function>(Init)) { if (Function *F = dyn_cast<Function>(Init)) {
nl(Out)<< "/ Function Declarations"; nl(Out); nl(Out)<< "/ Function Declarations"; nl(Out);
printFunctionHead(F); printFunctionHead(F);
} else if (GlobalVariable* gv = dyn_cast<GlobalVariable>(Init)) { } else if (GlobalVariable* gv = dyn_cast<GlobalVariable>(Init)) {
nl(Out) << "// Global Variable Declarations"; nl(Out); nl(Out) << "// Global Variable Declarations"; nl(Out);
printVariableHead(gv); printVariableHead(gv);
} else {
nl(Out) << "// Constant Definitions"; nl(Out);
printConstant(gv);
}
if (GlobalVariable* gv = dyn_cast<GlobalVariable>(Init)) {
nl(Out) << "// Global Variable Definitions"; nl(Out); nl(Out) << "// Global Variable Definitions"; nl(Out);
printVariableBody(gv); printVariableBody(gv);
} else {
nl(Out) << "// Constant Definitions"; nl(Out);
printConstant(Init);
} }
} }
} }