[PATCH] [LLD] COFF: Define error categories for Driver,	Reader and Resolver.
    Rui Ueyama 
    ruiu at google.com
       
    Sun May 31 16:05:23 PDT 2015
    
    
  
================
Comment at: COFF/Error.h:21
@@ +20,3 @@
+// Driver errors
+enum class DriverError { Success, InvalidOption };
+
----------------
rafael wrote:
> I don't think you need Success. You can just use std::error_code(), no?
Correct. I'll remove that.
Does it make sense to merge all the error categories into one LLDErrorCategory and let the caller distinguish error type by its value? What do you think?
http://reviews.llvm.org/D10129
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list