From 7acd886ecfa0adc8a14476eafe8cf1fa981cfe18 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Wed, 11 Jun 2014 19:05:50 +0000 Subject: Use std::error_code instead of llvm::error_code. The idea of this patch is to turn llvm/Support/system_error.h into a transitional header that just brings in the erorr_code api to the llvm namespace. I will remove it shortly afterwards. The cases where the general idea needed some tweaking: * std::errc is a namespace in msvc, so we cannot use "using std::errc". I could add an #ifdef, but there were not that many uses, so I just added std:: to them in this patch. * Template specialization had to be moved to the std namespace in this patch set already. * The msvc implementation of default_error_condition doesn't seem to provide the same transformations as we need. Not too surprising since the standard doesn't actually say what "equivalent" means. I fixed the problem by keeping our old mapping and using it at error_code construction time. Despite these shortcomings I think this is still a good thing. Some reasons: * The different implementations of system_error might improve over time. * It removes 925 lines of code from llvm already. * It removes 6313 bytes from the text segment of the clang binary when it is built with gcc and 2816 bytes when building with clang and libstdc++. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@210687 91177308-0d34-0410-b5e6-96231b3b80d8 --- tools/llvm-readobj/Error.cpp | 2 +- tools/llvm-readobj/Error.h | 6 ++++-- tools/llvm-readobj/ObjDumper.h | 4 ++-- tools/llvm-readobj/llvm-readobj.h | 2 -- 4 files changed, 7 insertions(+), 7 deletions(-) (limited to 'tools/llvm-readobj') diff --git a/tools/llvm-readobj/Error.cpp b/tools/llvm-readobj/Error.cpp index a68ebeb58e..a276fcedb8 100644 --- a/tools/llvm-readobj/Error.cpp +++ b/tools/llvm-readobj/Error.cpp @@ -50,7 +50,7 @@ std::string _readobj_error_category::message(int EV) const { error_condition _readobj_error_category::default_error_condition(int EV) const { if (static_cast(EV) == readobj_error::success) return error_condition(); - return errc::invalid_argument; + return std::errc::invalid_argument; } namespace llvm { diff --git a/tools/llvm-readobj/Error.h b/tools/llvm-readobj/Error.h index ebf652e9a1..b3aab327cb 100644 --- a/tools/llvm-readobj/Error.h +++ b/tools/llvm-readobj/Error.h @@ -33,8 +33,10 @@ inline error_code make_error_code(readobj_error e) { return error_code(static_cast(e), readobj_category()); } -template <> struct is_error_code_enum : std::true_type { }; - } // namespace llvm +namespace std { +template <> struct is_error_code_enum : std::true_type {}; +} + #endif diff --git a/tools/llvm-readobj/ObjDumper.h b/tools/llvm-readobj/ObjDumper.h index 9e0fd2f990..0c7eb5109f 100644 --- a/tools/llvm-readobj/ObjDumper.h +++ b/tools/llvm-readobj/ObjDumper.h @@ -10,6 +10,8 @@ #ifndef LLVM_READOBJ_OBJDUMPER_H #define LLVM_READOBJ_OBJDUMPER_H +#include "llvm/Support/system_error.h" + #include namespace llvm { @@ -18,8 +20,6 @@ namespace object { class ObjectFile; } -class error_code; - class StreamWriter; class ObjDumper { diff --git a/tools/llvm-readobj/llvm-readobj.h b/tools/llvm-readobj/llvm-readobj.h index cc5c85d8da..61ee0d4ed4 100644 --- a/tools/llvm-readobj/llvm-readobj.h +++ b/tools/llvm-readobj/llvm-readobj.h @@ -18,8 +18,6 @@ namespace llvm { class RelocationRef; } - class error_code; - // Various helper functions. bool error(error_code ec); bool relocAddressLess(object::RelocationRef A, -- cgit v1.2.3