[compiler-rt] DO_NOT_MERGE (PR #66682)
Vitaly Buka via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 20 14:17:55 PDT 2023
================
@@ -446,8 +458,51 @@ class BaseReport {
const HeapChunk heap;
const Allocations allocations;
const OverflowCandidate candidate;
+
+ struct {
+ uptr addr = 0;
+ tag_t tags[512] = {};
----------------
vitalybuka wrote:
we will align on 16, so we want the same number of lines before and after (17lines x 16bytes)
so logic will be more complicated, but possible.
However if we want to keep printing outside (we may want a print tags function outside of reporting), then I'd rather go with larger snapshot, but simpler logic.
Another issue is that when we do a snapshot we don't use a pierce address. in one of usecases we add access size as offset.
https://github.com/llvm/llvm-project/pull/66682
More information about the llvm-commits
mailing list