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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits