[PATCH] D33516: Add generic __bswap[ds]i2 implementations

Ed Maste via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 24 17:25:11 PDT 2017


emaste accepted this revision.
emaste added a comment.
This revision is now accepted and ready to land.

LGTM from my perspective.



================
Comment at: lib/builtins/bswapdi2.c:20-27
+	return ((((u) & 0xff00000000000000ULL) >> 56)
+	    | (((u) & 0x00ff000000000000ULL) >> 40)
+	    | (((u) & 0x0000ff0000000000ULL) >> 24)
+	    | (((u) & 0x000000ff00000000ULL) >>  8)
+	    | (((u) & 0x00000000ff000000ULL) <<  8)
+	    | (((u) & 0x0000000000ff0000ULL) << 24)
+	    | (((u) & 0x000000000000ff00ULL) << 40)
----------------
Does clang-format agree with this style?


https://reviews.llvm.org/D33516





More information about the llvm-commits mailing list