On Fri, Apr 4, 2014 at 12:27 PM, Ramana Radhakrishnan <ramana....@googlemail.com> wrote: > 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 ?
Ok. Thanks, Richard. > 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.
