On Mon, Sep 29, 2014 at 8:54 PM, Aldy Hernandez <al...@redhat.com> wrote: > I'm rearranging some code in Michael's original patch to minimize the > difference with mainline. > > It seems that the check for DECL_STRUCT_FUNCTION (decl)->gimple_df, was > merely a check to see if we had already set the FDE bits for the decl in > question.
Sounds more like a check whether the frontend is finished? > I've moved the check inside the original DECL_EXTERNAL check, > thus making it obvious what is being accomplished. > > I also got rid of mainline's gcc_checking_assert of fun. We're > dereferencing it immediately after. That should be enough to trigger an > ICE. > > Also I removed Michael's check for DECL_STRUCT_FUNCTION(decl), since > mainline drops into this codepath regardless, and has/had that > gcc_checking_assert anyhow. > > No regressions. > > Committed to branch. > > Aldy