10 Jul
2004
10 Jul
'04
1:36 p.m.
Frank Heckenbach wrote:
Waldek Hebisch wrote:
I had checked that 3.4 based cross assembler generates different output. However, in 3.4 backend does not append dots and numbers to the name if it has assembler name set, so my 3.4 patch skips setting assembler name on locals. So, probably with 3.4 it is enough to set the assembler name (as DECL_CONTEXT change was ineffective).
May be. But it shouldn't hurt to set DECL_CONTEXT to NULL_TREE, should it? So it's probably better to do so, so it works with all backends (as far as tested), and describes better what we actually do.
Agreed. I had to modify your patch for 3.4 (the 'set_assembler_name' hunk did not apply). After that all test (except for 2 that failed before) pass on AMD64.
--
Waldek Hebisch
hebisch@math.uni.wroc.pl