Hi Richard, Please review this revised patch. In this version, I've used #define for all duplicates except 'bextr' - that intrinsic is actually different between AMD and Intel. Test cases also updated to account for that difference.
Thanks! On Wed, Apr 30, 2014 at 8:39 PM, Richard Smith <[email protected]>wrote: > What a mess :( > > Can we define one in terms of the other, rather than duplicating the > definitions of all of these? (Maybe even #define one to the other?) > > > On Wed, Apr 30, 2014 at 3:14 PM, Sanjay Patel <[email protected]>wrote: > >> Please review (and commit if ok) the attached patch which duplicates all >> of the existing double-underscore BMI intrinsics with single-underscore >> variants. >> >> More details here: >> http://llvm.org/bugs/show_bug.cgi?id=19431 >> >> The equivalent bug in GCC is already fixed: >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60847 >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> >> > -- Sanjay Patel RotateRight, LLC http://www.rotateright.com
bmi_rev2.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
