Author: ed Date: Thu Mar 9 02:04:07 2017 New Revision: 297364 URL: http://llvm.org/viewvc/llvm-project?rev=297364&view=rev Log: Fix up the places where AddressSpace.hpp is included.
The AddressSpace.hpp header declares two classes: LocalAddressSpace and RemoteAddressSpace. These classes are only used in a very small number of source files, but passed in as template arguments to many other classes. Let's go ahead and only include AddressSpace.hpp in source files where at least one of these two classes is mentioned. This gets rid of a cyclic header dependency that was already present, but only caused breakage on macOS until recently. Reported by: Marshall Clow Modified: libunwind/trunk/src/CompactUnwinder.hpp libunwind/trunk/src/DwarfInstructions.hpp libunwind/trunk/src/DwarfParser.hpp libunwind/trunk/src/EHHeaderParser.hpp libunwind/trunk/src/Unwind_AppleExtras.cpp libunwind/trunk/src/libunwind.cpp Modified: libunwind/trunk/src/CompactUnwinder.hpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/CompactUnwinder.hpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/CompactUnwinder.hpp (original) +++ libunwind/trunk/src/CompactUnwinder.hpp Thu Mar 9 02:04:07 2017 @@ -19,7 +19,6 @@ #include <libunwind.h> #include <mach-o/compact_unwind_encoding.h> -#include "AddressSpace.hpp" #include "Registers.hpp" #define EXTRACT_BITS(value, mask) \ Modified: libunwind/trunk/src/DwarfInstructions.hpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/DwarfInstructions.hpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/DwarfInstructions.hpp (original) +++ libunwind/trunk/src/DwarfInstructions.hpp Thu Mar 9 02:04:07 2017 @@ -18,7 +18,6 @@ #include <stdlib.h> #include "dwarf2.h" -#include "AddressSpace.hpp" #include "Registers.hpp" #include "DwarfParser.hpp" #include "config.h" Modified: libunwind/trunk/src/DwarfParser.hpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/DwarfParser.hpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/DwarfParser.hpp (original) +++ libunwind/trunk/src/DwarfParser.hpp Thu Mar 9 02:04:07 2017 @@ -21,7 +21,6 @@ #include "libunwind.h" #include "dwarf2.h" -#include "AddressSpace.hpp" #include "config.h" namespace libunwind { Modified: libunwind/trunk/src/EHHeaderParser.hpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/EHHeaderParser.hpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/EHHeaderParser.hpp (original) +++ libunwind/trunk/src/EHHeaderParser.hpp Thu Mar 9 02:04:07 2017 @@ -15,7 +15,6 @@ #include "libunwind.h" -#include "AddressSpace.hpp" #include "DwarfParser.hpp" namespace libunwind { Modified: libunwind/trunk/src/Unwind_AppleExtras.cpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/Unwind_AppleExtras.cpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/Unwind_AppleExtras.cpp (original) +++ libunwind/trunk/src/Unwind_AppleExtras.cpp Thu Mar 9 02:04:07 2017 @@ -9,6 +9,7 @@ //===----------------------------------------------------------------------===// #include "config.h" +#include "AddressSpace.hpp" #include "DwarfParser.hpp" #include "unwind_ext.h" Modified: libunwind/trunk/src/libunwind.cpp URL: http://llvm.org/viewvc/llvm-project/libunwind/trunk/src/libunwind.cpp?rev=297364&r1=297363&r2=297364&view=diff ============================================================================== --- libunwind/trunk/src/libunwind.cpp (original) +++ libunwind/trunk/src/libunwind.cpp Thu Mar 9 02:04:07 2017 @@ -24,6 +24,7 @@ #include <stdlib.h> +#include "AddressSpace.hpp" #include "UnwindCursor.hpp" using namespace libunwind; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits