[PATCH] D10964: [Codegen] Add intrinsics 'hsum*' and corresponding SDNodes for horizontal sum operation.

Shahid via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 02:09:14 PDT 2015

ashahid added a comment.

Hi Hal,

Thanks for pointing this.

> Unsigned overflow is also undefined? Is this really necessary? I doubt that the vectorizer will be able to prove no-unsigned-overflow in most cases, and we don't get it from C's semantics, >and thus, would not be able to generate this intrinsic.

I referred to the C's semantics of "unsigned overflow" and realized that it is not necessary, will update the patch accordingly.



More information about the llvm-commits mailing list