[PATCH] error_code for dynamically generated error messages
Shankar Easwaran
shankar.kalpathi.easwaran at gmail.com
Thu May 22 18:32:33 PDT 2014
================
Comment at: lib/Core/Error.cpp:180
@@ +179,3 @@
+ llvm::sys::SmartScopedLock<true> lock(_mutex);
+ // Value zero is always the successs value.
+ if (_messages.empty())
----------------
Nit pick spell error of success
================
Comment at: lib/Core/Error.cpp:189
@@ +188,3 @@
+private:
+ std::vector<std::string> _messages;
+ llvm::sys::SmartMutex<true> _mutex;
----------------
How do we use the vector of messages ? Are these for predefined messages ? If so should it be a vector of StringRef ?
http://reviews.llvm.org/D3881
More information about the llvm-commits
mailing list