On Thu, Mar 27, 2014 at 11:25 AM, Ramana Radhakrishnan <ramra...@arm.com> wrote: > Hi, > > This is a partial fix for PR60655 where dwarf2out.c rejects NOT of a > value in const_ok_for_output_1. There is still a problem with the testcase > on armhf where we get operations of the form, const (minus (const_int) > (symref)) without the -fdata-sections option which is just weird. I'm not > yet sure where this is produced from and will not have the time to dig > further today. > > As Jakub said on IRC, const_ok_for_output_1 is called only with partial > rtx's and therefore disabling minus (const_int) (symref) might not be the > best thing to do especially if this were part of plus (symref) (minus (const > int) (symref)) and both symrefs were in the same section. I will try and > find sometime to investigate this further tomorrow. > > Bootstrapped and regtested on armhf > > Bootstrap and regression test running on x86_64. > > Ok to commit ?
Ping ? Ramana > > regards > Ramana > > gcc/ > > <DATE> Jakub Jelinek <ja...@redhat.com> > Ramana Radhakrishnan <ramana.radhakrish...@arm.com> > > * dwarf2out.c (const_ok_for_output_1): Reject expressions containing > a NOT. > > gcc/testsuite > > <DATE> Ramana Radhakrishnan <ramana.radhakrish...@arm.com> > > * gcc.c-torture/compile/pr60655-1.c: New test.
