Sorry I missed this earlier. I'm all for removing unnecessary builtins. Does the backend lower these shuffles to the same instruction the builtin does? Do we have tests for that in the llvm side?
Do you plan to remove the intrinsics from llvm in favor of the shuffles? The casts inside the diagnostic pragmas seem inconsistent between the two macros. Was that intentional. Is there already an emmintrin test file this could be added to? Maybe just put them in sse-builtins.c http://reviews.llvm.org/D7199 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
