Re: Remove hash from remember_with_vars

2013-09-02 Thread Jan Hubicka
Hi, unfortunately this patch ICEs on the following testcase /* This used to fail on SPARC with an unaligned memory access. */ void foo(int n) { struct S { int i[n]; unsigned int b:1; int i2; } __attribute__ ((packed)) __attribute__ ((aligned (4))); struct S s; s.i2 = 0; }

Remove hash from remember_with_vars

2013-08-31 Thread Jan Hubicka
Hi, remember_with_vars walks trees that are read from file (now unique) and looks for fields that can contain pointers to vars or functions and if so, it records them to global hashtable for later fixup. This is quite wasteful, because the hash is querried many times. We can simply walk all fields

Re: Remove hash from remember_with_vars

2013-08-31 Thread Richard Biener
Jan Hubicka hubi...@ucw.cz wrote: Hi, remember_with_vars walks trees that are read from file (now unique) and looks for fields that can contain pointers to vars or functions and if so, it records them to global hashtable for later fixup. This is quite wasteful, because the hash is querried many

Re: Remove hash from remember_with_vars

2013-08-31 Thread Bernhard Reutner-Fischer
On 31 August 2013 19:15:46 Richard Biener rguent...@suse.de wrote: Jan Hubicka hubi...@ucw.cz wrote: Hi, remember_with_vars walks trees that are read from file (now unique) and looks for fields that can contain pointers to vars or functions and if so, it records them to global hashtable for