This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, middle-end]: Fix PR 43057, [LTO] fold check: original tree changed by fold


On Mon, Nov 22, 2010 at 09:34:11AM +0100, Uros Bizjak wrote:
> Your proposed approach also works (and fixes bootstrap with
> --enable-checking=all, too). If there are no objections from Richi, I
> propose to commit this change to mainline (since it enables bootstrap
> with --enable-checking=fold) and fix all fold fails from the testsuite
> separately.

Fine with me; if Richi agrees to that, it would be better to put this into
a separate inline (fold-const.c local is ok) though, as it might be used
later on in multiple locations.

And then at the next phase get rid of all the protected_set_expr_location
and SET_EXPR_LOCATION uses in fold-const.c, either (when used on
the result of buildN), start using buildN_loc, or call this new inline
instead.

	Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]