ofiwg / libfabric

Open Fabric Interfaces
http://libfabric.org/
Other
547 stars 375 forks source link

rxm compiler warnings #3837

Closed jsquyres closed 6 years ago

jsquyres commented 6 years ago

With this morning's HEAD (d7a9a14a0996c3a0729371c52c6ddae353ad9c66), I see the following compiler warnings in rxm with gcc 7.3.0:

  CC       prov/rxm/src/src_libfabric_la-rxm_ep.lo
In file included from ./include/ofi_mem.h:41:0,
                 from ./include/ofi.h:51,
                 from prov/rxm/src/rxm_ep.c:35:
prov/rxm/src/rxm_ep.c: In function ‘rxm_ep_send_common’:
./include/ofi_list.h:72:13: warning: ‘tx_entry’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  item->prev = head;
  ~~~~~~~~~~~^~~~~~
prov/rxm/src/rxm_ep.c:968:23: note: ‘tx_entry’ was declared here
  struct rxm_tx_entry *tx_entry;
                       ^~~~~~~~
In file included from ./include/ofi_mem.h:41:0,
                 from ./include/ofi.h:51,
                 from prov/rxm/src/rxm_ep.c:35:
prov/rxm/src/rxm_ep.c: In function ‘rxm_ep_inject’:
./include/ofi_list.h:72:13: warning: ‘tx_entry’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  item->prev = head;
  ~~~~~~~~~~~^~~~~~
prov/rxm/src/rxm_ep.c:968:23: note: ‘tx_entry’ was declared here
  struct rxm_tx_entry *tx_entry;
                       ^~~~~~~~
In file included from ./include/ofi_mem.h:41:0,
                 from ./include/ofi.h:51,
                 from prov/rxm/src/rxm_ep.c:35:
prov/rxm/src/rxm_ep.c: In function ‘rxm_ep_tinjectdata’:
./include/ofi_list.h:72:13: warning: ‘tx_entry’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  item->prev = head;
  ~~~~~~~~~~~^~~~~~
prov/rxm/src/rxm_ep.c:968:23: note: ‘tx_entry’ was declared here
  struct rxm_tx_entry *tx_entry;
                       ^~~~~~~~
In file included from ./include/ofi_mem.h:41:0,
                 from ./include/ofi.h:51,
                 from prov/rxm/src/rxm_ep.c:35:
prov/rxm/src/rxm_ep.c: In function ‘rxm_ep_injectdata’:
./include/ofi_list.h:72:13: warning: ‘tx_entry’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  item->prev = head;
  ~~~~~~~~~~~^~~~~~
prov/rxm/src/rxm_ep.c:968:23: note: ‘tx_entry’ was declared here
  struct rxm_tx_entry *tx_entry;
                       ^~~~~~~~
In file included from ./include/ofi_mem.h:41:0,
                 from ./include/ofi.h:51,
                 from prov/rxm/src/rxm_ep.c:35:
prov/rxm/src/rxm_ep.c: In function ‘rxm_ep_tinject’:
./include/ofi_list.h:72:13: warning: ‘tx_entry’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  item->prev = head;
  ~~~~~~~~~~~^~~~~~
prov/rxm/src/rxm_ep.c:968:23: note: ‘tx_entry’ was declared here
  struct rxm_tx_entry *tx_entry;
                       ^~~~~~~~
dmitrygx commented 6 years ago

Thanks. This will be addressed here - #3822