ributzka added inline comments.

================
Comment at: clang/test/Modules/file-manager-lookup-count.m:73
+// RUN: FileCheck --input-file %t/stats %s
+// CHECK:      {
+// CHECK-NEXT:   "file-search.NumDirCacheMisses": 156,
----------------
The content of the stats file may change over time, so the test shouldn't limit 
it to just the listed entries. I would drop the first and last `CHECK` and 
change `CHECK-NEXT` to just `CHECK`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136019

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

Reply via email to