whisperity added a comment.
(Typing this in here so it is not forgotten.) @aaron.ballman suggested in
http://reviews.llvm.org/D69560#inline-893813 that a case of argument and
parameter swaps could be discovered between forward declarations and the
definitions, i.e.
void fn(int x, int y);
void fn(int y, int x) { ... } // Proparly mistakenly swapped names!
The conclusion was that this check (given this one does string distance,
metrics, etc.) should be extended with such functionality. However, we need a
better name for the check in that case!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D20689/new/
https://reviews.llvm.org/D20689
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
