Quuxplusone added inline comments.

================
Comment at: test/std/numerics/rand/rand.device/lit.local.cfg:1
+# Disable all of the random device tests if the correct feature is not 
available.
+if 'libcpp-has-no-random-device' in config.available_features:
----------------
EricWF wrote:
> There are only 3 tests under this directory. I would rather mark each one 
> explicitly with `// UNSUPPORTED: libcpp-has-no-random-device`
@weimingz: looks like this request from Eric is unresolved.

FWIW: I'm still ambivalent about the whole direction of this patch; but I'm 
happy if everyone else is happy. I don't anticipate any new comments from me.


https://reviews.llvm.org/D41316



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

Reply via email to