summaryrefslogtreecommitdiffstats
path: root/llvm/lib
diff options
context:
space:
mode:
authorEhsan Akhgari <ehsan.akhgari@gmail.com>2015-08-19 02:05:37 +0000
committerEhsan Akhgari <ehsan.akhgari@gmail.com>2015-08-19 02:05:37 +0000
commitc62c6396103c30a49f7e9cb0fa3ba657816c77ba (patch)
tree600878886142f616be0072e3265873a80a92594f /llvm/lib
parenta8d205f14504b13448892081cc19a9accb0d566c (diff)
downloadbcm5719-llvm-c62c6396103c30a49f7e9cb0fa3ba657816c77ba.tar.gz
bcm5719-llvm-c62c6396103c30a49f7e9cb0fa3ba657816c77ba.zip
Insert override at the same line as the end of the function declaration
Summary: The existing check converts the code pattern below: void f() { } to: void f() override { } which is fairly sub-optimal. This patch fixes this by inserting the override keyword on the same line as the function declaration if possible, so that we instead get: void f() override { } We do this by looking for the last token before the start of the body and inserting the override keyword at the end of its location. Note that we handle const, volatile and ref-qualifiers correctly. Test Plan: Includes an automated test. Reviewers: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D9286 llvm-svn: 245401
Diffstat (limited to 'llvm/lib')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud