xbolva00 added inline comments.

================
Comment at: libcxx/include/bit:211
+        ? __t
+        : (__t << (__cnt % __dig)) | (__t >> (__dig - (__cnt % __dig)));
+}
----------------
mclow.lists wrote:
> Quuxplusone wrote:
> > No sane compiler would actually generate three `mod` operations for the 
> > three instances of repeated subexpression `__cnt % __dig`, would they?
> At `-O0`? Sure it would.
Hoist it? 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D51262/new/

https://reviews.llvm.org/D51262



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to