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 <[email protected]> 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:* [email protected] [mailto: > [email protected]] *On Behalf Of *Yin Ma > *Sent:* Monday, August 13, 2012 5:55 PM > *To:* [email protected] > *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 **** > > ** ** >
2012_08_31-10_18_03_0001-Add-support-for-fvector-select.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
