
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">

</head>

<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">

<br>

<div>

<div>On Mar 11, 2013, at 11:27 AM, Nadav Rotem <<a href="mailto:nrotem@apple.com">nrotem@apple.com</a>></div>

<div> wrote:</div>

<br class="Apple-interchange-newline">

<blockquote type="cite">

<div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;">

<blockquote type="cite">

<div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;">

<br class="Apple-interchange-newline">

Line 4501 in trunk DAGCombiner.cpp… I changed the ISD::SELECT to the VT.isVector() ? ISD::VSELECT : ISD::SELECT...</div>

<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;">

</blockquote>

<div><br>

</div>

<div>Thanks. From the commit message I think that we should only run this optimization on scalars. </div>

</div>

</blockquote>

<div><br>

</div>

<div>I have tried making just that change… it definitely helps quite a bit. I submitted the simple patch to llvm-commits for approval.</div>

<div><br>

</div>

<div>It doesn't quite clear up ALL the asserts I see in my test runs… I'm still getting the assert in ExpandVSELECT in a few cases, but I suppose the custom lowering is the "easy" way to solve those, though I'm sure the TRUNCATE would do the trick as well.</div>

<div><br>

</div>

<div>Thanks for your help.</div>

<div><br>

</div>

<div>Richard</div>

</div>

</body>

</html>