phosek added inline comments.

================
Comment at: clang/test/CodeGen/profile-filter.c:17
+// RUN: %clang_cc1 -fprofile-instrument=clang 
-fprofile-list=%t-exclude-only.list -emit-llvm %s -o - | FileCheck %s 
--check-prefix=EXCLUDE
+
+unsigned i;
----------------
davidxl wrote:
> phosek wrote:
> > davidxl wrote:
> > > Can you add a test case with both include and exclude list (the 
> > > intersection of INCLUDE_SET & !EXCLUDE_SET)?
> > See the case above this one which has:
> > ```
> > fun:test*
> > !fun:test1
> > ```
> > Is that what you have in mind?
> Right
Do we need to do anything then since this test case already exists?


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

https://reviews.llvm.org/D94820

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

Reply via email to