[cfe-commits] [PATCH][Review place] A Patch to Enable Vector Select for arm general computing

[Stephen Hines]

Ping for review.

Thanks,
Steve


On Fri, Aug 31, 2012 at 10:27 AM, Stephen Hines <srhines at google.com> wrote:

> Hi Yin (and cfe-commits),
>
> I reworked this patch so that it is not dependent on ARM. I added
> -fvector-select (and the corresponding LangOpts.VectorSelect) to enable
> this functionality. I also updated the test to work with non-ARM targets.
> Can CFE-commits take another look and submit if it is acceptable?
>
> Thanks,
> Steve
>
>
> On Thu, Aug 30, 2012 at 1:34 PM, Yin Ma <yinma at codeaurora.org> wrote:
>
>> Hi Stephen,****
>>
>> ** **
>>
>>      The attached file are the latest diff file to enable vector select
>> for ARM and ****
>>
>> A test case that should be placed in test/Sema/ directory. The code has
>> been****
>>
>> rebased on the latest clang source. Please give a review. ****
>>
>> ** **
>>
>> Thanks,****
>>
>> ** **
>>
>>                             Yin ****
>>
>> ** **
>>
>> ** **
>>
>> *From:* cfe-commits-bounces at cs.uiuc.edu [mailto:
>> cfe-commits-bounces at cs.uiuc.edu] *On Behalf Of *Yin Ma
>> *Sent:* Monday, August 13, 2012 5:55 PM
>> *To:* cfe-commits at cs.uiuc.edu
>> *Subject:* [cfe-commits] [PATCH][Review place] A Patch to Enable Vector
>> Select for general computing****
>>
>> ** **
>>
>> Hi,****
>>
>> ** **
>>
>> So far vector select, such as  a < b ? a : c is  not enabled in clang for
>> C/C++ code.****
>>
>> It only enabled for OpenCL. I have made a very small change in clang to
>> enable****
>>
>> Vector select. I have tested for ARM.  Everything works fine so far. ****
>>
>>    The file attached enables vector select for general computing. Please
>> give ****
>>
>> A review.****
>>
>> ** **
>>
>> Thanks,****
>>
>> ** **
>>
>>                Yin ****
>>
>> ** **
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120905/6cb6ed8e/attachment.html>