[llvm-bugs] [Bug 7417] fixits need to be de-duped somehow

[via llvm-bugs]

https://llvm.org/bugs/show_bug.cgi?id=7417

Richard Smith <richard-llvm at metafoo.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |richard-llvm at metafoo.co.uk
         Resolution|FIXED                       |---

--- Comment #13 from Richard Smith <richard-llvm at metafoo.co.uk> ---
Fixits should be deduplicated, but this patch is not an appropriate direction
for doing so. If we want to support fixits in templates, we should have a
centralized mechanism for deduplication rather than working around this in an
ad-hoc, per-fixit way.

Also, this approach of modifying the template during instantiation is unsound
(it doesn't work if the AST node is reused in instantiations of the template,
the change doesn't get propagated into a module file built from the
compilation, it does the wrong thing if the same AST node is used in multiple
places, we modified the AST even in non-error cases such as SFINAE or the MS
extension path, ...).

Reverted in r249342. If we want to do this, we should use a centralized
mechanism that deduplicates identical fixits (and maybe even identical
diagnostics in general) from different instantiations of the same template, or
similar.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20151005/d059fd09/attachment.html>