<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>I am not sure what is the right approach here. Another alternative is to use the TTI interface. We can wither add a new API for checking if the target benefits from vectorization or we can use this API to check how many vector registers are available:</div><div><br></div><div><pre class="fragment" style="font-family: Fixed, monospace; font-size: 13px; border: 1px solid rgb(204, 204, 204); background-color: rgb(245, 245, 245); margin: 4px 8px 4px 2px; padding: 4px 6px; position: static; z-index: auto;"><span class="keywordtype" style="color: rgb(96, 64, 32);">unsigned</span> TargetVectorRegisters = TTI.getNumberOfRegisters(<span class="keyword" style="color: rgb(0, 128, 0);">true</span>);
</pre></div><div><br></div><div>Thanks,</div><div>Nadav</div><div><br></div><div><br></div><div><div>On Sep 6, 2013, at 10:34 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">I think the idea is OK. Nadav, any objections?<br><br>On the patch itself:<br><br>hasVectorizer is a strange name. disableVectorizerByDefault or<br>something else without a "has" would probably be better.<br><br>The two uses of hasVectorizer are in a single function. Can't you put<br>its return valuue in a variable and reuse it?<br><br><br>On 6 September 2013 05:39, Robert Lytton <<a href="mailto:robert@xmos.com">robert@xmos.com</a>> wrote:<br><blockquote type="cite">Is it OK to go?<br><br>________________________________<br>From: <a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a> [<a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a>] on<br>behalf of Robert Lytton [<a href="mailto:robert@xmos.com">robert@xmos.com</a>]<br>Sent: 02 September 2013 14:26<br>To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>Subject: XCore target: disable vectorization<br><br>Hi,<br><br>Here is a patch to disable vectorization for a target by default (only<br>xcore).<br><br>Thank you<br><br>Robert<br><br><br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br><br></blockquote></blockquote></div><br></body></html>