[PATCH] D147287: [InstrProf] Temporal Profiling

Ellis Hoag via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 17:57:00 PDT 2023


ellis added inline comments.


================
Comment at: llvm/lib/ProfileData/InstrProfWriter.cpp:440
   Header.BinaryIdOffset = 0;
+  Header.FunctionTracesOffset = 0;
   int N = sizeof(IndexedInstrProf::Header) / sizeof(uint64_t);
----------------
snehasish wrote:
> ellis wrote:
> > snehasish wrote:
> > > There seem to be a few different phrases referring to the same functionality, e.g. `TemporalProfile`, `FunctionTraces` and just `Trace` in other places. It would be nice to have a single prefix for all the var and function names to be able to quickly navigate the code. What do you think?
> > I see what you mean. I'm using `FunctionTrace` and `Trace` interchangeably and they aren't very descriptive. What do you think of the `TP` (Temporal Profiling) prefix? So I would use `TPTrace` in code and `:tp_traces` for the text format.
> I prefer something more verbose like "TemporalProf", but up to you to decide, "TP" as the prefix is fine too.
Thanks for the feedback! I ended up going with `TemporalProf...` and I tried to update all the variables to be more consistent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147287



More information about the llvm-commits mailing list