[PATCH] D103550: [SampleFDO] New hierarchical discriminator for FS SampleFDO (llvm-profdata part)

[Hongtao Yu via Phabricator via llvm-commits]

hoy accepted this revision.
hoy added inline comments.


================
Comment at: llvm/tools/llvm-profdata/llvm-profdata.cpp:572
   auto Reader = std::move(ReaderOrErr.get());
+  setDiscriminatorMaskedBit(Reader.get());
   if (std::error_code EC = Reader->read())
----------------
xur wrote:
> hoy wrote:
> > I'm wondering if this can be moved into `SampleProfileReader::create` with the mask as an extra parameter of it.
> That's a good idea -- I think this would simplify the code. Let me do the change.
Thanks for working on it. LGTM.


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

https://reviews.llvm.org/D103550