On Mon, Aug 19, 2013 at 11:44 AM, Jeff Law <l...@redhat.com> wrote: > On 08/19/2013 12:34 PM, Teresa Johnson wrote: >> >> Ping. >> Thanks, >> Teresa >> >> On Sun, Aug 11, 2013 at 9:35 PM, Teresa Johnson <tejohn...@google.com> >> wrote: >>> >>> This patch fixes PR rtl-optimizations/57451 by preventing scopes and >>> therefore lexical blocks from crossing split section boundaries. >>> This will prevent debug info generation from using DW_AT_low_pc/high_pc >>> pairs across the section boundary. >>> >>> Bootstrapped and tested on x86_64-unknown-linux-gnu. With this patch, >>> a profilebootstrap with -freorder-blocks-and-partition force-enabled >>> also passes. Ok for trunk? >>> >>> Thanks, >>> Teresa >>> >>> 2013-08-11 Teresa Johnson <tejohn...@google.com> >>> >>> PR rtl-optimizations/57451 >>> * final.c (reemit_insn_block_notes): Prevent lexical blocks >>> from crossing split section boundaries. > > This is fine. Is there any way you can turn the steps mentioned in 57451 > into a dejagnu testcase? Clearly it would only work in a native > configuration due to the need to run the instrumented program.
Since this was a compile-time failure (link-time actually), I could just take the existing test case mentioned there and turn it into a "-g -freorder-blocks-and-partition" test (probably by cloning as a new test case the g++.dg/tree-prof testsuite directory). Does that seem reasonable? Thanks, Teresa > > jeff > -- Teresa Johnson | Software Engineer | tejohn...@google.com | 408-460-2413
