[PATCH] D114400: [SampleFDO] Compute BFI if the sample loader changes BPI in FSAFDO

[Wenlei He via Phabricator via llvm-commits]

wenlei added inline comments.


================
Comment at: llvm/lib/CodeGen/MIRSampleProfile.cpp:318
+  if (Changed)
+    MBFI->calculate(MF, *MBFI->getMBPI(), *&getAnalysis<MachineLoopInfo>());
 
----------------
When profile loader leads to change, we should probably recalculate BFI. But should BFI calculation be done outside of sample loader given BFI is a separate analysis? The IR BFI is done outside of sample loader. 

It feels to me that updating MBFI inside sample loader kind of breaks the componentization. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114400