PR28106, build of 2.37 fails on FreeBSD and Clang

https://en.cppreference.com/w/cpp/types/NULL says NULL might be
defined as nullptr.
https://en.cppreference.com/w/cpp/language/reinterpret_cast says
reinterpret_cast can't be used on nullptr.

	PR gold/28106
	PR gold/27815
	* gc.h (gc_process_relocs): Use static_cast in Section_id constructor.

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 6aba1e0e3ec5ca7c5f1a162ec6ab1674df6b4f76..df92052da7dd850baf64d4ba08e2e37434319da1 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,9 @@
+2021-07-20  Alan Modra  <amodra@gmail.com>
+
+       PR gold/28106
+       PR gold/27815
+       * gc.h (gc_process_relocs): Use static_cast in Section_id constructor.
+
 2021-07-17  Michael Krasnyk  <michael.krasnyk@gmail.com>
 
        PR gold/28098

diff --git a/gold/gc.h b/gold/gc.h
index 6ebd81f3fd0eebfd3459c96fa5c0dbd0451c2e46..4a1368f5abe4d54171de76e31df574477a504916 100644 (file)
--- a/gold/gc.h
+++ b/gold/gc.h
@@ -247,7 +247,7 @@ gc_process_relocs(
              if (is_ordinary) 
                (*secvec).push_back(Section_id(src_obj, dst_indx));
              else
-               (*secvec).push_back(Section_id(reinterpret_cast<Relobj*>(NULL), 0));
+               (*secvec).push_back(Section_id(static_cast<Relobj*>(NULL), 0));
               // If the target of the relocation is an STT_SECTION symbol,
               // make a note of that by storing -1 in the symbol vector.
               if (lsym.get_st_type() == elfcpp::STT_SECTION)
@@ -329,7 +329,7 @@ gc_process_relocs(
               if (is_ordinary && dst_obj != NULL)
                (*secvec).push_back(Section_id(dst_obj, dst_indx));
              else
-               (*secvec).push_back(Section_id(reinterpret_cast<Relobj*>(NULL), 0));
+               (*secvec).push_back(Section_id(static_cast<Relobj*>(NULL), 0));
               (*symvec).push_back(gsym);
              (*addendvec).push_back(std::make_pair(
                                        static_cast<long long>(symvalue),