[cfe-dev] clang-apply-replacements not de-duplicating replacements

Malcolm Parsons via cfe-dev cfe-dev at lists.llvm.org
Wed Mar 6 03:45:19 PST 2019


On 05/03/2019 01:43, Riyaz Puthiyapurayil via cfe-dev wrote:
> I have the following YAML file generated from clang-tidy (test-case at
> the end of the email). Notice that there are two identical replacements
> /u/riyaz/test.cc at offset:117. I am aware that clang-tidy does NOT
> de-duplicate replacements. But I thought clang-apply-replacements was
> supposed to do it. However, it adds the replacement text (override
> keyword) @offset:117 twice.

Which version of clang-apply-replacements are you referring to?

clang-apply-replacements does de-duplicate replacements in all
branches except release_70.
See https://reviews.llvm.org/rL342951

-- 
Malcolm Parsons



More information about the cfe-dev mailing list