[PATCH] D70553: [clang-apply-replacements] Add command line option to overwrite readonly files.

Zachary Turner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 6 10:22:00 PST 2019


zturner added a comment.

In D70553#1757862 <https://reviews.llvm.org/D70553#1757862>, @aaron.ballman wrote:

> Can you add a test case for this functionality?


The reason there's no test case is because it seems like that wouldn't really be any different than testing the functionality of `llvm::sys::fs`, which should already be handled at the llvm/Support layer.  There's not really any interesting logic here.  I can still do one if you think it's important but that was my reasoning anyway.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D70553





More information about the cfe-commits mailing list