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

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 27 09:33:36 PDT 2022


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.



================
Comment at: clang/test/CodeGen/pgo-sample-use-profi.cpp:1
+//Test if profi flat is enabled in frontend as user-facing feature.
+//
----------------
Remove the test.

This change only needs a clang/test/Driver test showing that the option passes `-sample-profile-use-profi` to -cc1.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136846



More information about the cfe-commits mailing list