[PATCH] D135914: [PseudoProbe] Decode offset based pseudo probe.

[Wenlei He via Phabricator via llvm-commits]

wenlei added a comment.

Sentinel probe case isn't covered for both encoding and decoding in any test cases, would be good to have some coverage.



================
Comment at: llvm/lib/MC/MCPseudoProbe.cpp:418
   // Decide if top-level node should be disgarded.
-  if (Cur == &DummyInlineRoot && !GuildFilter.empty() &&
-      !GuildFilter.count(Guid))
+  if (Cur == &DummyInlineRoot && !GuidFilter.empty() && !GuidFilter.count(Guid))
     Cur = nullptr;
----------------
nit: `Cur == &DummyInlineRoot` -> `IsTopLevelFunc`


================
Comment at: llvm/lib/MC/MCPseudoProbe.cpp:476
+        // leading probe address or function start address.
+        EncodingIsAddrBased = true;
+      }
----------------
Maybe I'm asking the same question as the one on previous patch, but why do we still have cases with absolute address encoded? I thought for main function body we have GUID -> symbol (reverse lookup) -> address; and we have similar way to get address for sentinel for split binary function. The rest should all be offset/delta.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135914