[PATCH] D148305: remove useless call in MIRSampleProfile.cpp

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 16 23:40:12 PDT 2023


skan added inline comments.


================
Comment at: llvm/lib/CodeGen/MIRSampleProfile.cpp:277
   ProfileIsValid = (Reader->read() == sampleprof_error::success);
-  Reader->getSummary();
 
----------------
I'm afraid we can not remove it b/c `getSummary` may change the internal state of `Reader`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148305



More information about the llvm-commits mailing list