[PATCH] D63482: [clang-tidy] Fix the YAML created for checks like modernize-pass-by-value

Matthias Gehre via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 4 04:14:26 PDT 2020


mgehre added a comment.

Could be - did it handle it correctly for your include case here?

And if this is a general yaml string thing, shouldn't the replacement of newlines (for both ways) happen in the yaml parser/writer instead of clang/Tooling?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63482





More information about the cfe-commits mailing list