ellis added inline comments.
================ Comment at: llvm/lib/ProfileData/InstrProfCorrelator.cpp:208 maybeSwap<IntPtrT>(CounterOffset), + maybeSwap<IntPtrT>(BitmaskOffset), maybeSwap<IntPtrT>(FunctionPtr), ---------------- Since `BitmaskOffset` and `NumBitmaskBytes` are always zero, I would rather set them as zero here rather than passing in more parameters. See `ValuesPtr`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136385/new/ https://reviews.llvm.org/D136385 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits