[PATCH] D136846: [Driver] Add -fsample-profile-use-profi

Zhang Haoyu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 6 18:05:17 PST 2022


HaoyuZhang added a comment.

Thanks @MaskRay and @wenlei for the reviewing.

New version for using dummy file has been updated.



================
Comment at: clang/test/Driver/pgo-sample-use-profi.c:2
+/// Test if profi flat is enabled in frontend as user-facing feature.
+// RUN: %clang -c -fsample-profile-use-profi -fprofile-sample-use=%S/../CodeGen/Inputs/pgo-sample.prof -### %s 2>&1 | FileCheck %s
+
----------------
MaskRay wrote:
> Such a cross-directory test file reference is generally not acceptable. Use a dummy file `-fprofile-sample-use=/dev/null`
Thank you for the comments. Modified.


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