[PATCH] D74448: [scudo][standalone] Allow setting release to OS

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 09:08:56 PST 2020


cryptoad added a comment.

Running some tests, 1 comment.



================
Comment at: compiler-rt/lib/scudo/standalone/wrappers_c.inc:165
+        value = INT32_MIN;
+        secondary_interval = INT32_MIN;
+      } else {
----------------
Now that value & secondary_interval have the same values, does it make sense to merge them?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74448





More information about the llvm-commits mailing list