XCore target: disable vectorization

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Sep 6 10:34:33 PDT 2013


I think the idea is OK. Nadav, any objections?

On the patch itself:

hasVectorizer is a strange name. disableVectorizerByDefault or
something else without a "has" would probably be better.

The two uses of hasVectorizer are in a single function. Can't you put
its return valuue in a variable and reuse it?


On 6 September 2013 05:39, Robert Lytton <robert at xmos.com> wrote:
> Is it OK to go?
>
> ________________________________
> From: cfe-commits-bounces at cs.uiuc.edu [cfe-commits-bounces at cs.uiuc.edu] on
> behalf of Robert Lytton [robert at xmos.com]
> Sent: 02 September 2013 14:26
> To: cfe-commits at cs.uiuc.edu
> Subject: XCore target: disable vectorization
>
> Hi,
>
> Here is a patch to disable vectorization for a target by default (only
> xcore).
>
> Thank you
>
> Robert
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list