[PATCH] Add return to DIType::Verify
Renato Golin
renato.golin at linaro.org
Tue Nov 26 04:55:14 PST 2013
Hi echristo,
Code scanner ran by Sylvestre Ledru got a no_return bug in DebugInfo.cpp. Adding the return statements that should be there.
http://llvm-reviews.chandlerc.com/D2271
Files:
lib/IR/DebugInfo.cpp
Index: lib/IR/DebugInfo.cpp
===================================================================
--- lib/IR/DebugInfo.cpp
+++ lib/IR/DebugInfo.cpp
@@ -461,11 +461,11 @@
// DIType is abstract, it should be a BasicType, a DerivedType or
// a CompositeType.
if (isBasicType())
- DIBasicType(DbgNode).Verify();
+ return DIBasicType(DbgNode).Verify();
else if (isCompositeType())
- DICompositeType(DbgNode).Verify();
+ return DICompositeType(DbgNode).Verify();
else if (isDerivedType())
- DIDerivedType(DbgNode).Verify();
+ return DIDerivedType(DbgNode).Verify();
else
return false;
return true;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2271.1.patch
Type: text/x-patch
Size: 634 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131126/d15b5b08/attachment.bin>
More information about the llvm-commits
mailing list