Szelethus marked 3 inline comments as done. Szelethus added inline comments.
================ Comment at: test/Analysis/Inputs/expected-plists/plist-macros-with-expansion.cpp.plist:6 + <key>clang_version</key> +<string>clang version 8.0.0 (http://mainstream.inf.elte.hu/Szelethus/clang 85a6dda64587a5a18482f091cbcf020fbd3ec1dd) (https://github.com/llvm-mirror/llvm 1ffbf26a1a0a190d69327af875a3337b74a2ce82)</string> + <key>diagnostics</key> ---------------- whisperity wrote: > You are diffing against a hardcoded output file which contains //a git > hash//! Won't this break at the next commit?! I'm actually using `%diff_plist`! ``` // Check the actual plist output. // RUN: cat %t.plist | %diff_plist \ // RUN: %S/Inputs/expected-plists/plist-macros-with-expansion.cpp.plist ``` Here's the patch that implemented it (should jump to the config file): D52036#change-mOw5EmE2OiVA Repository: rC Clang https://reviews.llvm.org/D52742 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
