2014-12-09 0:00 GMT+03:00 Jeff Law <l...@redhat.com>: > On 12/08/14 04:00, Ilya Enkovich wrote: >>> >>> 2014-11-26 Ilya Enkovich <ilya.enkov...@intel.com> >>> >>> PR bootstrap/63995 >>> * tree-chkp.c (chkp_make_static_bounds): Share bounds var >>> between nodes sharing assembler name. >>> >>> gcc/testsuite >>> >>> 2014-11-26 Ilya Enkovich <ilya.enkov...@intel.com> >>> >>> PR bootstrap/63995 >>> * g++.dg/dg.exp: Add mpx-dg.exp. >>> * g++.dg/pr63995-1.C: New. > > Code seems fine. Not sure how the testcase is actually testing that you're > sharing bounds though. The test AFAICT just verifies that it can be > compiled and doesn't checkout the output in any way. Or is it the case that > the test will ICE or segfault without your fix?
Without this fix compiler emits two symbols with the same name and assembler complains. Something like that: /tmp/cceHGHa1.s: Assembler messages: /tmp/cceHGHa1.s:12080: Error: symbol `__chkp_bounds_of_mode_inner' is already defined /tmp/cceHGHa1.s:12086: Error: symbol `__chkp_bounds_of_mode_nunits' is already defined Thanks, Ilya > > > Jeff > >
