hubert.reinterpretcast added a comment. In D62952#1548593 <https://reviews.llvm.org/D62952#1548593>, @aaron.ballman wrote:
> But is there a reason to not keep `%diff_sarif` and define it in terms of > `%normalize_sarif | diff -U1 -b` within lit.local.cfg? I guess I don't see > the benefit to exposing the call to diff (I don't anticipate anyone needing > to change the options passed to diff). Yes: The normalization runs on stdin. `%diff_sarif` would then only make sense if stdin is one of the inputs, and I think the only input we can hardcode to be stdin is the one conventionally considered to be the reference input (which is not what we want). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62952/new/ https://reviews.llvm.org/D62952 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
