[llvm] r199622 - [PM] Fix a contradiction in the comments noticed by Anders.
Chandler Carruth
chandlerc at gmail.com
Sun Jan 19 18:32:02 PST 2014
Author: chandlerc
Date: Sun Jan 19 20:32:02 2014
New Revision: 199622
URL: http://llvm.org/viewvc/llvm-project?rev=199622&view=rev
Log:
[PM] Fix a contradiction in the comments noticed by Anders.
Have I mentioned that functions returning true on error and false on
success are confusing? They're more confusing when their name is
"verify". Anyways...
Modified:
llvm/trunk/include/llvm/IR/Verifier.h
Modified: llvm/trunk/include/llvm/IR/Verifier.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/Verifier.h?rev=199622&r1=199621&r2=199622&view=diff
==============================================================================
--- llvm/trunk/include/llvm/IR/Verifier.h (original)
+++ llvm/trunk/include/llvm/IR/Verifier.h Sun Jan 19 20:32:02 2014
@@ -34,14 +34,14 @@ class raw_ostream;
/// pass.
///
/// If there are no errors, the function returns false. If an error is found,
-/// a message describing the error is written to OS (if non-null) and false is
+/// a message describing the error is written to OS (if non-null) and true is
/// returned.
bool verifyFunction(const Function &F, raw_ostream *OS = 0);
/// \brief Check a module for errors.
///
/// If there are no errors, the function returns false. If an error is found,
-/// a message describing the error is written to OS (if non-null) and false is
+/// a message describing the error is written to OS (if non-null) and true is
/// returned.
bool verifyModule(const Module &M, raw_ostream *OS = 0);
More information about the llvm-commits
mailing list