Fix copy-pasto, allocate objfile_per_bfd_storage with obstack_new
authorSimon Marchi <simon.marchi@polymtl.ca>
Mon, 21 May 2018 03:18:29 +0000 (23:18 -0400)
committerSimon Marchi <simon.marchi@polymtl.ca>
Mon, 21 May 2018 03:19:35 +0000 (23:19 -0400)
commit184cde7552b5434196b8380be23b39ff4a5a17e6
tree81ae1abf58389b739753cd699e39695d4400848c
parente39db4db7c553ae1c4aaf158cd0ebf3cf6d478fb
Fix copy-pasto, allocate objfile_per_bfd_storage with obstack_new

I realized after pushing that I made a copy-pasto, I had:

  # define HAVE_IS_TRIVIALLY_COPYABLE 1

instead of

  # define HAVE_IS_TRIVIALLY_CONSTRUCTIBLE 1

with the consequence that IsMallocable was always std::true_type (and
was therefore not enforcing anything).  Fixing that mistake triggered a
build failure:

/home/simark/src/binutils-gdb/gdb/objfiles.c:150:12:   required from here
/home/simark/src/binutils-gdb/gdb/common/poison.h:228:3: error: static assertion failed: Trying to use XOBNEW with a non-POD data type.

I am not sure why I did not see this when I originally wrote the patch
(but I saw and fixed other failures).  In any case, I swapped XOBNEW
with obstack_new to get rid of it.

Regtested on the buildbot.

gdb/ChangeLog:

* common/traits.h (HAVE_IS_TRIVIALLY_COPYABLE): Rename the wrong
instance to...
(HAVE_IS_TRIVIALLY_CONSTRUCTIBLE): ... this.
* objfiles.c (get_objfile_bfd_data): Allocate
objfile_per_bfd_storage with obstack_new when allocating on
obstack.
gdb/ChangeLog
gdb/common/traits.h
gdb/objfiles.c