This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Oh dear. I regret to inform you that commit 'RAII-fy make_cleanup_restore_current_thread & friends' might be unfortunate


On 05/04/2017 07:28 PM, Pedro Alves wrote:
> On 05/04/2017 07:22 PM, Pedro Alves wrote:
> 
>> AFAICS so far, this is a false positive.
>>
>> Not sure what to do.  I wouldn't want to force-memset
>> the optional's storage to work around it, which would be
>> a pessimization to quiet a warning.  From above, we see that
>> that wouldn't work when we later start using std::optional.
>>
>> There's a bug open about this (for boost::optional, but most
>> probably the exact same):
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78044
> 
> I've tried the reproducer there with the obvious change to use
> std::optional and that does not warn.  So this is a different,
> though related issue.
> 
> Or maybe there's really something wrong with the gdb code
> that is escaping me.

OK, here's smallest, self-contained reproducer I managed to
come up with:

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

//#include <optional>
#include <new>

template<typename T>
struct optional
{
  optional () : m_dummy () {}
  ~optional () { m_item.~T (); }
  void emplace () { new (&m_item) T (); }

  union
  {
    int m_dummy;
    T m_item;
  };
};

template <typename T>
using Optional = optional<T>; // warns
//using Optional = std::optional<T>; // warns too

extern int get ();
extern void set (int);

struct A
{
  A () : m (get ()) {}
  ~A () { set (m); }

  int m;
};

struct B
{
  B ();
  ~B ();
};

void func ()
{
  Optional<A> maybe_a;
  Optional<B> maybe_b;

  maybe_a.emplace ();
  maybe_b.emplace ();
}
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


$ /opt/gcc/bin/g++ optional.cc -g3 -O2 -Wmaybe-uninitialized -std=gnu++17 -c
optional.cc: In function ‘void func()’:
optional.cc:28:15: warning:
‘maybe_a.optional<A>::<anonymous>.optional<A>::<unnamed union>::m_dummy’
may be used uninitialized in this function [-Wmaybe-uninitialized]
   ~A () { set (m); }
           ~~~~^~~
optional.cc:41:15: note:
‘maybe_a.optional<A>::<anonymous>.optional<A>::<unnamed union>::m_dummy’
was declared here
   Optional<A> maybe_a;
               ^~~~~~~

Looks like a compiler bug to me.

Thanks,
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]