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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to