jfb abandoned this revision.
jfb added a comment.
Herald added a subscriber: dang.
I don't think the world is ready for this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77219/new/
https://reviews.llvm.org/D77219
jfb marked an inline comment as done.
jfb added inline comments.
Comment at: compiler-rt/lib/ubsan_bel/ubsan_bel_handlers.cpp:40-42
+static std::random_device r;
+static std::default_random_engine e(r());
+static std::uniform_int_distribution d(0, sizeof(quips) /
siz
lebedev.ri added inline comments.
Comment at: compiler-rt/lib/ubsan_bel/ubsan_bel_handlers.cpp:40-42
+static std::random_device r;
+static std::default_random_engine e(r());
+static std::uniform_int_distribution d(0, sizeof(quips) /
sizeof(quips[0]) - 1);
---
regehr added a comment.
Sorry but I don't think this can land until it has options for sending
sanitizer output to Slack channels and SMS numbers.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77219/new/
https://reviews.llvm.org/D77219
jfb created this revision.
Herald added subscribers: Sanitizers, cfe-commits, ributzka, dexonsmith,
jkorous, cryptoad, mgorny.
Herald added projects: clang, Sanitizers.
jfb edited the summary of this revision.
Yes, this is April 1st and the patch isn't particularly serious.
There is a ␇ UBSan ru