[PATCH] D136846: [Driver] Enable profi flag as user-facing flag

2022-10-27 Thread Wenlei He via Phabricator via cfe-commits
wenlei added a comment. I'm curious why do you need profi to be a driver flag? There're many similar tuning flags that don't have corresponding driver flag. profi is narrower than most tuning flags as it's specific to one component (profile inference) unique to sample PGO. Repository: rG LL

[PATCH] D136846: [Driver] Enable profi flag as user-facing flag

2022-10-27 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay requested changes to this revision. MaskRay added a comment. This revision now requires changes to proceed. Herald added a subscriber: StephenFan. > [Driver] Enable profi flag as user-facing flag Please avoid unclear term "profi" and use "-fsample-profile-use-profi" instead. ==

[PATCH] D136846: [Driver] Enable profi flag as user-facing flag

2022-10-27 Thread Zhang Haoyu via Phabricator via cfe-commits
HaoyuZhang created this revision. Herald added a subscriber: wenlei. Herald added a project: All. HaoyuZhang requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. This patch enable -sample-profile-use-profi in Clang frontend as user-facin