dexonsmith added inline comments.
================ Comment at: test/Preprocessor/headermap-rel.c:2-3 // This uses a headermap with this entry: // Foo.h -> Foo/Foo.h ---------------- Should we delete this comment now that the header map is human-readable? ================ Comment at: test/Preprocessor/headermap-rel.c:6 +// RUN: rm -f %t.hmap +// RUN: hmaptool write %S/Inputs/headermap-rel/foo.hmap.json %t.hmap +// RUN: %clang_cc1 -E %s -o %t.i -I %t.hmap -F %S/Inputs/headermap-rel ---------------- Should there be some sort of `REQUIRES:` clause for using `hmaptool`? https://reviews.llvm.org/D46485 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
