[llvm] r175380 - The transform is:

[Bill Wendling]

On Feb 16, 2013, at 4:06 PM, Hal Finkel <hfinkel at anl.gov> wrote:

> ----- Original Message -----
>> From: "Bill Wendling" <isanbard at gmail.com>
>> To: llvm-commits at cs.uiuc.edu
>> Sent: Saturday, February 16, 2013 5:41:36 PM
>> Subject: [llvm] r175380 - The transform is:
>> 
>> Author: void
>> Date: Sat Feb 16 17:41:36 2013
>> New Revision: 175380
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=175380&view=rev
>> Log:
>> The transform is:
>> 
>>    (or (bool?A:B),(bool?C:D)) --> (bool?(or A,C):(or B,D))
>> 
>> By the time the OR is visited, both the SELECTs have been visited and
>> not
>> optimized and the OR itself hasn't been transformed so we do this
>> transform in
>> the hopes that the new ORs will be optimized.
>> 
>> The transform is explicitly disabled for vector-selects until
>> "codegen matures
>> to handle them better".
> 
> I thought that Nadav had said that vector select support was better now. What happened with that?
> 
It's possible that Muhammad Tauqir wasn't aware of it. I'll let him revamp his patch if he wishes. :)

-bw