From: Ian Lance Taylor Date: Thu, 3 Apr 2008 14:18:37 +0000 (+0000) Subject: * compressed_output.h (class Output_compressed_section): Use X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=126f3ece278737f6cac83e7e6f251579cc0e4eee;p=binutils-gdb.git * compressed_output.h (class Output_compressed_section): Use unsigned buffer. * compressed_output.cc (zlib_compress): Use unsigned buffers, add zlib header. (zlib_compressed_suffix): Removed. (Output_compressed_section::set_final_data_size): Use unsigned buffers. * testsuite/Makefile.am (flagstest_compress_debug_sections): Fix linker invocation. (flagstest_o_specialfile_and_compress_debug_sections): Likewise. * testsuite/Makefile.in: Regenerated. --- diff --git a/gold/ChangeLog b/gold/ChangeLog index 53b055fd2f7..ecd4e5582a1 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,18 @@ +2008-04-03 Craig Silverstein + + * compressed_output.h (class Output_compressed_section): Use + unsigned buffer. + * compressed_output.cc (zlib_compress): Use unsigned buffers, + add zlib header. + (zlib_compressed_suffix): Removed. + (Output_compressed_section::set_final_data_size): Use unsigned + buffers. + * testsuite/Makefile.am (flagstest_compress_debug_sections): + Fix linker invocation. + (flagstest_o_specialfile_and_compress_debug_sections): + Likewise. + * testsuite/Makefile.in: Regenerated. + 2008-04-02 David S. Miller * dwarf_reader.cc (Sized_dwarf_line_info::read_header_prolog, diff --git a/gold/Makefile.in b/gold/Makefile.in index 127888fd738..55adee1ee8d 100644 --- a/gold/Makefile.in +++ b/gold/Makefile.in @@ -50,7 +50,7 @@ DIST_COMMON = README $(am__configure_deps) $(srcdir)/../config.guess \ $(srcdir)/../mkinstalldirs $(srcdir)/Makefile.am \ $(srcdir)/Makefile.in $(srcdir)/config.in \ $(top_srcdir)/configure $(top_srcdir)/po/Make-in ChangeLog \ - NEWS pread.c yyscript.c yyscript.h + NEWS TODO pread.c yyscript.c yyscript.h subdir = . ACLOCAL_M4 = $(top_srcdir)/aclocal.m4 am__aclocal_m4_deps = $(top_srcdir)/../config/depstand.m4 \ diff --git a/gold/compressed_output.cc b/gold/compressed_output.cc index c7452333607..a0f8ed181ac 100644 --- a/gold/compressed_output.cc +++ b/gold/compressed_output.cc @@ -38,15 +38,21 @@ namespace gold // (including not having zlib support in the library). If it returns // true, it allocates memory for the compressed data using new, and // sets *COMPRESSED_DATA and *COMPRESSED_SIZE to appropriate values. +// It also writes a header before COMPRESSED_DATA: 4 bytes saying +// "ZLIB", and 8 bytes indicating the uncompressed size, in big-endian +// order. #ifdef HAVE_ZLIB_H static bool -zlib_compress(const char* uncompressed_data, unsigned long uncompressed_size, - char** compressed_data, unsigned long* compressed_size) +zlib_compress(const unsigned char* uncompressed_data, + unsigned long uncompressed_size, + unsigned char** compressed_data, + unsigned long* compressed_size) { + const int header_size = 12; *compressed_size = uncompressed_size + uncompressed_size / 1000 + 128; - *compressed_data = new char[*compressed_size]; + *compressed_data = new unsigned char[*compressed_size + header_size]; int compress_level; if (parameters->options().optimize() >= 1) @@ -54,13 +60,19 @@ zlib_compress(const char* uncompressed_data, unsigned long uncompressed_size, else compress_level = 1; - int rc = compress2(reinterpret_cast(*compressed_data), + int rc = compress2(reinterpret_cast(*compressed_data) + header_size, compressed_size, reinterpret_cast(uncompressed_data), uncompressed_size, compress_level); if (rc == Z_OK) - return true; + { + memcpy(*compressed_data, "ZLIB", 4); + elfcpp::Swap_unaligned<64, true>::writeval(*compressed_data + 4, + uncompressed_size); + *compressed_size += header_size; + return true; + } else { delete[] *compressed_data; @@ -72,24 +84,14 @@ zlib_compress(const char* uncompressed_data, unsigned long uncompressed_size, #else // !defined(HAVE_ZLIB_H) static bool -zlib_compress(const char*, unsigned long, char**, unsigned long*) +zlib_compress(const unsigned char*, unsigned long, + unsigned char**, unsigned long*) { return false; } #endif // !defined(HAVE_ZLIB_H) -// After compressing an output section, we rename it from foo to -// foo.zlib.nnnn, where nnnn is the uncompressed size of the section. - -static std::string -zlib_compressed_suffix(unsigned long uncompressed_size) -{ - char size_string[64]; - snprintf(size_string, sizeof(size_string), "%lu", uncompressed_size); - return std::string(".zlib.") + size_string; -} - // Class Output_compressed_section. // Set the final data size of a compressed section. This is where @@ -102,8 +104,7 @@ Output_compressed_section::set_final_data_size() // (Try to) compress the data. unsigned long compressed_size; - unsigned char* u_uncompressed_data = this->postprocessing_buffer(); - char* uncompressed_data = reinterpret_cast(u_uncompressed_data); + unsigned char* uncompressed_data = this->postprocessing_buffer(); // At this point the contents of all regular input sections will // have been copied into the postprocessing buffer, and relocations @@ -117,8 +118,8 @@ Output_compressed_section::set_final_data_size() &this->data_, &compressed_size); if (success) { - std::string suffix(zlib_compressed_suffix(uncompressed_size)); - this->new_section_name_ = std::string(this->name()) + suffix; + // This converts .debug_foo to .zdebug_foo + this->new_section_name_ = std::string(".z") + (this->name() + 1); this->set_name(this->new_section_name_.c_str()); this->set_data_size(compressed_size); } diff --git a/gold/compressed_output.h b/gold/compressed_output.h index 300b064c337..11b2762288f 100644 --- a/gold/compressed_output.h +++ b/gold/compressed_output.h @@ -64,7 +64,7 @@ class Output_compressed_section : public Output_section // The options--this includes the compression type. const General_options* options_; // The compressed data. - char* data_; + unsigned char* data_; // The new section name if we do compress. std::string new_section_name_; }; diff --git a/gold/testsuite/Makefile.am b/gold/testsuite/Makefile.am index 70b3e15dda7..d478ed45d20 100644 --- a/gold/testsuite/Makefile.am +++ b/gold/testsuite/Makefile.am @@ -507,7 +507,7 @@ undef_symbol.err: undef_symbol_main.o undef_symbol.so gcctestdir/ld # Test --compress-debug-sections. FIXME: check we actually compress. check_PROGRAMS += flagstest_compress_debug_sections flagstest_compress_debug_sections: flagstest_debug.o gcctestdir/ld - $(CXXLINK) -Bgcctestdir/ -o $@ $< --compress-debug-sections=zlib + $(CXXLINK) -Bgcctestdir/ -o $@ $< -Wl,--compress-debug-sections=zlib test -s $@ @@ -523,7 +523,7 @@ flagstest_o_specialfile: flagstest_debug.o gcctestdir/ld check_PROGRAMS += flagstest_o_specialfile_and_compress_debug_sections flagstest_o_specialfile_and_compress_debug_sections: flagstest_debug.o \ gcctestdir/ld - $(CXXLINK) -Bgcctestdir/ -o /dev/stdout $< --compress-debug-sections=zlib 2>&1 | cat > $@ + $(CXXLINK) -Bgcctestdir/ -o /dev/stdout $< -Wl,--compress-debug-sections=zlib 2>&1 | cat > $@ chmod a+x $@ test -s $@ diff --git a/gold/testsuite/Makefile.in b/gold/testsuite/Makefile.in index c0eefbb48b7..dfa6bfb259d 100644 --- a/gold/testsuite/Makefile.in +++ b/gold/testsuite/Makefile.in @@ -1813,7 +1813,7 @@ uninstall-am: uninstall-info-am @GCC_TRUE@@NATIVE_LINKER_TRUE@ exit 1; \ @GCC_TRUE@@NATIVE_LINKER_TRUE@ fi @GCC_TRUE@@NATIVE_LINKER_TRUE@flagstest_compress_debug_sections: flagstest_debug.o gcctestdir/ld -@GCC_TRUE@@NATIVE_LINKER_TRUE@ $(CXXLINK) -Bgcctestdir/ -o $@ $< --compress-debug-sections=zlib +@GCC_TRUE@@NATIVE_LINKER_TRUE@ $(CXXLINK) -Bgcctestdir/ -o $@ $< -Wl,--compress-debug-sections=zlib @GCC_TRUE@@NATIVE_LINKER_TRUE@ test -s $@ @GCC_TRUE@@NATIVE_LINKER_TRUE@flagstest_o_specialfile: flagstest_debug.o gcctestdir/ld @GCC_TRUE@@NATIVE_LINKER_TRUE@ $(CXXLINK) -Bgcctestdir/ -o /dev/stdout $< 2>&1 | cat > $@ @@ -1821,7 +1821,7 @@ uninstall-am: uninstall-info-am @GCC_TRUE@@NATIVE_LINKER_TRUE@ test -s $@ @GCC_TRUE@@NATIVE_LINKER_TRUE@flagstest_o_specialfile_and_compress_debug_sections: flagstest_debug.o \ @GCC_TRUE@@NATIVE_LINKER_TRUE@ gcctestdir/ld -@GCC_TRUE@@NATIVE_LINKER_TRUE@ $(CXXLINK) -Bgcctestdir/ -o /dev/stdout $< --compress-debug-sections=zlib 2>&1 | cat > $@ +@GCC_TRUE@@NATIVE_LINKER_TRUE@ $(CXXLINK) -Bgcctestdir/ -o /dev/stdout $< -Wl,--compress-debug-sections=zlib 2>&1 | cat > $@ @GCC_TRUE@@NATIVE_LINKER_TRUE@ chmod a+x $@ @GCC_TRUE@@NATIVE_LINKER_TRUE@ test -s $@ @GCC_TRUE@@NATIVE_LINKER_TRUE@ver_test_1.so: ver_test_1.o ver_test_2.so ver_test_3.o ver_test_4.so gcctestdir/ld