[PATCH] D34731: [Sanitizers] Operator new() interceptors always die on allocation error

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 22:40:52 PDT 2017


cryptoad added inline comments.


================
Comment at: lib/scudo/scudo_new_delete.cpp:33
+  void *res = scudoMalloc(size, FromNew);
+  if (!res) DieOnFailure::OnOOM();
+  return res;
----------------
Thanks for doing those changes on Scudo as well.
Would it possible to get mark the failure branches as `UNLIKELY`? (at least the ones in Scudo).


https://reviews.llvm.org/D34731





More information about the llvm-commits mailing list