> On 1 June 2013 14:06, Jan Hubicka <hubi...@ucw.cz> wrote: > > Hi, > > this patch cleanups way we handle aliases. The main point is to merge code > > that was previously done separately for variables and functions. > > > > Hello, This patch appears to break both arm and aarch64. I don't > fully understand the mechanism. The issue shows up when building > res_libc.o over in glibc. The following fragment demonstrates the > issue: > > I suspect the section anchor appeared because > varasm.c:use_blocks_for_decl_p test for an 'alias' failed.
You are probably right. I added code that removes alias attributes once they land symtab. The reason is that I run into wrong code issues with code picking up the symbol name from alias attribute that is no longer valid after LTO renaming. Apparently this code and also symbol equality testing in fold-const rely on alias attribute but do not worry about what alias it is. I will revert this part of patch and try to solve this incrementally. (the code in fold-const is wrong anyway) Honza > > /Marcus
