[PATCH] D30569: [clang-tidy] misc-use-after-move: Fix failing assertion
Alexander Kornienko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 3 05:27:24 PST 2017
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG. Thanks!
================
Comment at: test/clang-tidy/misc-use-after-move.cpp:285
// CHECK-MESSAGES: [[@LINE-1]]:3: warning: 'a' used after it was moved
- // CHECK-MESSAGES: [[@LINE-3]]:6: note: move occurred here
+ // CHECK-MESSAGES: [[@LINE-3]]:7: note: move occurred here
}
----------------
mboehme wrote:
> mboehme wrote:
> > I've seen this column number flapping back and forth between 6 and 7 before.
> >
> > This seems to be unrelated to my fix and instead is triggered simply by adding the extra test case below; even with my fix present, this reverts back to 7 if I remove the test case. Looks like an obscure bug in the way the column number is computed...
> Should be "reverts back to 6"
Interesting. Are you sure there's nothing else here in play? Like non-ascii characters or weird line endings?
https://reviews.llvm.org/D30569
More information about the cfe-commits
mailing list