[PATCH] D68047: [SampleFDO] Create a separate flag ProfileAccurateForProfSymList for handling profile symbol list

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: lib/Transforms/IPO/SampleProfile.cpp:1674
 
-  if (ProfileSampleAccurate) {
+  if (ProfileAccurateForSymsInList) {
     NamesInProfile.clear();
----------------
check PSL too?


================
Comment at: lib/Transforms/IPO/SampleProfile.cpp:1782
+  // about new and hot functions being mistakenly treated as cold.
+  if (ProfileAccurateForSymsInList && PSL) {
+    // Initialize the entry count to 0 for functions in the list.
----------------
If symbol list is available, it will override the decision of -fprofile-sample-accurate. Is this the intended behavior?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68047