Page MenuHomePhabricator

Fix clang-tidy to account of correct source location of defaulted/deleted members
ClosedPublic

Authored by eliben on Mar 19 2015, 2:37 PM.

Details

Summary

Followup to http://reviews.llvm.org/D8465, which would break a test in clang-tidy. clang-tidy previously assumes that source locations of defaulted/deleted members are (incorrectly) not including the '= ...' part.

Diff Detail

Repository
rL LLVM

Event Timeline

eliben updated this revision to Diff 22304.Mar 19 2015, 2:37 PM
eliben retitled this revision from to Fix clang-tidy to account of correct source location of defaulted/deleted members.
eliben updated this object.
eliben edited the test plan for this revision. (Show Details)
eliben added reviewers: alexfh, sbenza.
eliben added a subscriber: Unknown Object (MLST).
alexfh accepted this revision.Mar 19 2015, 6:31 PM
alexfh edited edge metadata.

Looks good!

This revision is now accepted and ready to land.Mar 19 2015, 6:31 PM
sbenza accepted this revision.Mar 20 2015, 6:21 AM
sbenza edited edge metadata.

Thanks for the fix!

This revision was automatically updated to reflect the committed changes.

r233032.

Thanks for the review!