<feed xmlns='http://www.w3.org/2005/Atom'>
<title>bcm5719-llvm/clang-tools-extra/clang-tidy/readability, branch meklort-10.0.1</title>
<subtitle>Project Ortega BCM5719 LLVM</subtitle>
<id>https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.1</id>
<link rel='self' href='https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.1'/>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/'/>
<updated>2020-02-03T15:09:12+00:00</updated>
<entry>
<title>[clang-tidy] Fixed crash 44745 in readability-else-after-return</title>
<updated>2020-02-03T15:09:12+00:00</updated>
<author>
<name>Nathan James</name>
<email>n.james93@hotmail.co.uk</email>
</author>
<published>2020-02-02T14:04:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=7ad47b46b55fe33bc763411ae7b8bcca5a63d8dc'/>
<id>urn:sha1:7ad47b46b55fe33bc763411ae7b8bcca5a63d8dc</id>
<content type='text'>
Summary: Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=44745 | readability-else-after-return crashes ]]

Reviewers: aaron.ballman, alexfh, hokein, JonasToth, gribozavr2

Reviewed By: alexfh

Subscribers: merge_guards_bot, xazax.hun, cfe-commits

Tags: #clang, #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D73841

(cherry picked from commit d591bdce6d623208d4aeb335a762d839f0f8f0f7)
</content>
</entry>
<entry>
<title>[clang-tidy] Disable Checks on If constexpr statements in template Instantiations for BugproneBranchClone and ReadabilityBracesAroundStatements</title>
<updated>2020-01-22T23:57:56+00:00</updated>
<author>
<name>Nathan</name>
<email>n.james93@hotmail.co.uk</email>
</author>
<published>2019-12-31T09:57:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=1f98c2b586e4ebce68d75856699059663a052cb7'/>
<id>urn:sha1:1f98c2b586e4ebce68d75856699059663a052cb7</id>
<content type='text'>
Summary: fixes [[ https://bugs.llvm.org/show_bug.cgi?id=32203 | readability-braces-around-statements broken for if constexpr]] and [[ https://bugs.llvm.org/show_bug.cgi?id=44229 | bugprone-branch-clone false positive with template functions and constexpr ]] by disabling the relevant checks on if constexpr statements while inside an instantiated template. This is due to how the else branch of an if constexpr statement is folded away to a null statement if the condition evaluates to false

Reviewers: alexfh, hokein, aaron.ballman, xazax.hun

Reviewed By: aaron.ballman, xazax.hun

Subscribers: rnkovacs, JonasToth, Jim, lebedev.ri, xazax.hun, cfe-commits

Tags: #clang-tools-extra, #clang

Differential Revision: https://reviews.llvm.org/D71980

(cherry picked from commit f9c46229e4ac29053747c96e08c574c6c48d544b)
</content>
</entry>
<entry>
<title>Added readability-qualified-auto check</title>
<updated>2020-01-14T19:06:46+00:00</updated>
<author>
<name>Nathan James</name>
<email>n.james93@hotmail.co.uk</email>
</author>
<published>2020-01-14T19:05:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=36fcbb838c8f293f46bfed78c6ed8c177f1e3485'/>
<id>urn:sha1:36fcbb838c8f293f46bfed78c6ed8c177f1e3485</id>
<content type='text'>
Adds a check that detects any auto variables that are deduced to a pointer or
a const pointer then adds in the const and asterisk according. Will also
check auto L value references that could be written as const. This relates
to the coding standard
https://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto
</content>
</entry>
<entry>
<title>Fix readability-identifier-naming missing member variables</title>
<updated>2020-01-13T18:28:55+00:00</updated>
<author>
<name>Nathan James</name>
<email>n.james93@hotmail.co.uk</email>
</author>
<published>2020-01-13T18:26:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=fb79ef524171c96a9f3df025ac7a8a3e00fdc0b4'/>
<id>urn:sha1:fb79ef524171c96a9f3df025ac7a8a3e00fdc0b4</id>
<content type='text'>
Fixes PR41122 (missing fixes for member variables in a destructor) and
PR29005 (does not rename class members in all locations).
</content>
</entry>
<entry>
<title>[APFloat] Fix checked error assert failures</title>
<updated>2020-01-09T07:42:32+00:00</updated>
<author>
<name>Ehud Katz</name>
<email>ehudkatz@gmail.com</email>
</author>
<published>2020-01-09T07:42:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=24b326cc610dfdccdd50bc78505ec228d96c8e7a'/>
<id>urn:sha1:24b326cc610dfdccdd50bc78505ec228d96c8e7a</id>
<content type='text'>
`APFLoat::convertFromString` returns `Expected` result, which must be
"checked" if the LLVM_ENABLE_ABI_BREAKING_CHECKS preprocessor flag is
set.
To mark an `Expected` result as "checked" we must consume the `Error`
within.
In many cases, we are only interested in knowing if an error occured,
without the need to examine the error info. This is achieved, easily,
with the `errorToBool()` API.
</content>
</entry>
<entry>
<title>[clang-tidy] Disable match on `if constexpr` statements in template instantiation for `readability-misleading-indentation` check.</title>
<updated>2020-01-08T14:36:13+00:00</updated>
<author>
<name>Andi-Bogdan Postelnicu</name>
<email>abpostelnicu@me.com</email>
</author>
<published>2020-01-07T09:44:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=ba129c7d0f5c7c32398ad708c88e14cb06a339ad'/>
<id>urn:sha1:ba129c7d0f5c7c32398ad708c88e14cb06a339ad</id>
<content type='text'>
Summary: Fixes fixes `readability-misleading-identation` for `if constexpr`. This is very similar to D71980.

Reviewers: alexfh

Subscribers: xazax.hun, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D72333
</content>
</entry>
<entry>
<title>[APFloat] Fix compilation warnings</title>
<updated>2020-01-06T09:30:40+00:00</updated>
<author>
<name>Ehud Katz</name>
<email>ehudkatz@gmail.com</email>
</author>
<published>2020-01-06T08:51:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=f3f7dc3d2990151a78b246a7a1485d0c13a9fb36'/>
<id>urn:sha1:f3f7dc3d2990151a78b246a7a1485d0c13a9fb36</id>
<content type='text'>
</content>
</entry>
<entry>
<title>NFC: Fix trivial typos in comments</title>
<updated>2020-01-04T15:28:41+00:00</updated>
<author>
<name>Kazuaki Ishizaki</name>
<email>ishizaki@jp.ibm.com</email>
</author>
<published>2020-01-04T15:28:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=b7ecf1c1c373c53183ef6ef66efbe4237ff7b96d'/>
<id>urn:sha1:b7ecf1c1c373c53183ef6ef66efbe4237ff7b96d</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Handle init statements in readability-else-after-return</title>
<updated>2020-01-02T18:39:27+00:00</updated>
<author>
<name>Nathan James</name>
<email>n.james93@hotmail.co.uk</email>
</author>
<published>2020-01-02T18:37:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=ec3d8e61b527c6312f77a4dab095ffc34e954927'/>
<id>urn:sha1:ec3d8e61b527c6312f77a4dab095ffc34e954927</id>
<content type='text'>
Adds a new ASTMatcher condition called 'hasInitStatement()' that matches if,
switch and range-for statements with an initializer. Reworked clang-tidy
readability-else-after-return to handle variables in the if condition or init
statements in c++17 ifs. Also checks if removing the else would affect object
lifetimes in the else branch.

Fixes PR44364.
</content>
</entry>
<entry>
<title>[NFC] Fixes -Wrange-loop-analysis warnings</title>
<updated>2020-01-01T19:01:37+00:00</updated>
<author>
<name>Mark de Wever</name>
<email>koraq@xs4all.nl</email>
</author>
<published>2020-01-01T16:23:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=8dc7b982b4556c243e0502e6e230bdd53ddd65ff'/>
<id>urn:sha1:8dc7b982b4556c243e0502e6e230bdd53ddd65ff</id>
<content type='text'>
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to -Wall.

Differential Revision: https://reviews.llvm.org/D71857
</content>
</entry>
</feed>
