lgtm with comment

================
Comment at: lib/Headers/Intrin.h:419
@@ -418,2 +418,3 @@
 unsigned __int64 _shlx_u64(unsigned __int64, unsigned int);
 unsigned __int64 shrx_u64(unsigned __int64, unsigned int);
+static __inline__ unsigned __int64 __attribute__((__always_inline__, 
__nodebug__))
----------------
While you're here, is shrx_u64 missing an underscore?

================
Comment at: lib/Headers/Intrin.h:423
@@ +422,3 @@
+         unsigned __int64 *_HighProduct) {
+  __int128 FullProduct =
+      (unsigned __int128)_Multiplier * (unsigned __int128)_Multiplicand;
----------------
use an underscore for FullProduct to be safe?

http://reviews.llvm.org/D6486



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to