summaryrefslogtreecommitdiffstats
path: root/llvm/lib/CodeGen/XRayInstrumentation.cpp
diff options
context:
space:
mode:
authorAlexander Kornienko <alexfh@google.com>2017-05-22 13:58:16 +0000
committerAlexander Kornienko <alexfh@google.com>2017-05-22 13:58:16 +0000
commita0ddf5fc1c16e89ee1b382a0597050182f9f5a99 (patch)
treed9efb78121e81c5f3795d806789b1d75c383ade4 /llvm/lib/CodeGen/XRayInstrumentation.cpp
parent68ae83e68c6f1f874c162c9bcf032bf305724bab (diff)
downloadbcm5719-llvm-a0ddf5fc1c16e89ee1b382a0597050182f9f5a99.tar.gz
bcm5719-llvm-a0ddf5fc1c16e89ee1b382a0597050182f9f5a99.zip
[clang-tidy] readability-braces-around-statements false positive with char literals
Summary: Single-line if statements cause a false positive when the last token in the conditional statement is a char constant: ``` if (condition) return 'a'; ``` For some reason `findEndLocation` seems to skips too many (vertical) whitespaces in this case. The same problem already occured with string literals (https://reviews.llvm.org/D25558), and was fixed by adding a special check for this very case. I just extended the condition to also include char constants. No idea what really causes the issue though. Reviewers: alexfh Reviewed By: alexfh Subscribers: xazax.hun, cfe-commits Patch by Florian Gross! Differential Revision: https://reviews.llvm.org/D33354 llvm-svn: 303551
Diffstat (limited to 'llvm/lib/CodeGen/XRayInstrumentation.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud