[PATCH] D77219: UBSan ␇ runtime

[JF Bastien via Phabricator via cfe-commits]

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<int> d(0, sizeof(quips) / sizeof(quips[0]) - 1);
----------------
lebedev.ri wrote:
> I'm not sure we should be introducing randomness into UB handler.
> I think what this instead should do, is attach gdb to the process and fix the UB-invoking instructions.
Hmm, should it be sudo-random to ensure that `rm -rf /` works?

What is gdb? Is it like lldb but not as low-level?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77219