On Mon, Jan 09, 2017 at 06:25:05PM +0100, Andreas Tobler wrote: > On 09.01.17 12:25, Jakub Jelinek wrote: > > On Mon, Jan 09, 2017 at 11:53:38AM +0100, Richard Biener wrote: > > > > Ok, attached the part I bootstrapped successfully on amd64-*-freebsd12 > > > > and > > > > aarch64-*-freebsd12. From the amd64 run you'll find some test results > > > > at the > > > > usual place. The aarch64 run takes some more time. > > > > > > > > I hope I got it right this time :) > > > > What do you think? > > > > > > Looks good to me with the added comment to dwarf2out_late_global_decl > > > exchanged to the one on trunk. > > > > The formatting is completely wrong. > > Lines indented e.g. by 7 spaces (or tab + 1/3 space(s)), > > /* comment inside of { block starting in the same column as { > > (should be 2 columns to the right), && ! not aligned below VAR_P, > > or indenting by 3 columns instead of 2. > > Hehe, yep. This time done with emacs ;) > > Here the hopefully final patch with proper ChangeLog and formatting fixed. > > Ok to apply?
Formatting LGTM, so I think Richard's approval applies now. Jakub