[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