spatel added a comment. In https://reviews.llvm.org/D47202#1108016, @lebedev.ri wrote:
> That is what happens for "hand-rolled" `abs`, https://godbolt.org/g/6dbgXD > I *think* this makes sense, since **IIRC** LLVM only supports > twos-complement platforms. > But all these attributes are currently beyond me, so i won't click the > accept button :) Thanks for taking a look! I should also note that we don't translate the standard libcall to the builtin in clang (not sure if we should), so this patch doesn't currently affect the common case. We'd probably want to do something similar in LibCallSimplifier if this is right. https://reviews.llvm.org/D47202 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
