jhen added a comment.

I just found and fixed another bug in this patch. Before, I wasn't using the 
spelling location for the fixit hint. This meant that a macro argument that was 
expanded to two locations, for example, would have the same fixit hint applied 
to it twice. My new test case verifies that this does not happen anymore.


https://reviews.llvm.org/D25450



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to