efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land.
Let's do this one step at a time; first this patch, then fix the __builtin_* functions to use "e", then add all the missing cases CodeGenFunction::EmitBuiltinExpr. This patch LGTM, assuming you fix the commit message (and the title on Phabricator) to properly describe the change. https://reviews.llvm.org/D39204 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits