0005-Search-all-dominated-blocks-for-expressions-to-hoist.patch
Steven Bosscher
stevenb.gcc@gmail.com
Wed Jun 30 12:14:00 GMT 2010
On Wed, Jun 30, 2010 at 8:23 AM, Maxim Kuvyrkov <maxim@codesourcery.com> wrote:
> On 6/30/10 1:08 AM, Steven Bosscher wrote:
> ...
>>
>> The ideal patch would remove TRANSPOUT and clean out those expressions
>> earlier, see what RTL PRE does.
>
> Removing TRANSPOUT would be a separate change and there's no need to mash it
> together with this patch.
Indeed. This is why I said:
"If you clean that up first, this latest version 0003 patch will
probably look better/simpler."
Note the word "first".
But it's only a suggestion. It just seems to me that doing this would
simplify the verification of the implementation of your new algorithm
a bit easier. And that'd be a good thing...
Ciao!
Steven
More information about the Gcc-patches
mailing list