On 12/08/14 14:12, Ilya Enkovich wrote:
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
Ah OK. Ok for the trunk.
Thanks for your patience,
Jeff