On Thu, Aug 13, 2015 at 3:32 AM, Abe <abe_skol...@yahoo.com> wrote:
> Dear all,
>
> Overall, I think the WIP new if converter is holding up
> relatively well, but there is obviously opportunity to do better,
> at least if the numbers mean what they look like they mean,
> i.e. the old converter`s code was fully OK and so is the new one`s.
> By "fully OK" I mean e.g. no crashing bugs were introduced by the
> conversion.
>
>
> In the following, all the integers over 1000 are loops-vectorized counts.
>
>
> "base": baseline compiler source code
>         Git hash: cb791e75379bc0c8b10bd13bcb24305c36fd504b
>         commit date: July 10 2015
>         committer: Richard
>
> "new": base + patches for new [GIMPLE-level] if converter
>
>
>
> -O3
> ===
>
> no special flags
> ----------------
> base: 5951
> new:  5956
>
> with only "-ftree-loop-if-convert" added

That is -ftree-loop-if-convert-stores?

> ----------------------------------------
> base: 5954
> new:  5956
>
> with both if-conversion flags added

What's the other if-conversion flag?  I suppose _this_ is
-ftree-loop-if-convert-stores?  That would match the numbers
above which are mostly identical because -O3/-Ofast already
enable -ftree-loop-if-convert by means of enabling vectorization.

> -----------------------------------
> base: 5970
> new:  5956
>
>
>
> -Ofast
> ======
>
> no special flags
> ----------------
> base: 7393
> new:  7401
>
> with only "-ftree-loop-if-convert" added
> ----------------------------------------
> base: 7393
> new:  7401
>
> with both if-conversion flags added
> -----------------------------------
> base: 7421
> new:  7401
>

Can you please post individual benchmark numbers instead of just the
overall score?

>From the numbers above I can see the new if-converter removes any improvement
we get from -ftree-loop-if-convert-stores (as expected - it's not a
vectorization enabler
with the new scheme).

Thanks,
Richard.

>
> I have a spreadsheet [and a PDF generated therefrom] that shows the above in
> a
> more visual format.  Please feel free to ask for the PDF as an email
> attachment.
>
>
> Regards,
>
> Abe
>
>

Reply via email to