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