[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