diff options
| author | Yang Fan <nullptr.cpp@gmail.com> | 2021-04-09 10:22:56 +0800 |
|---|---|---|
| committer | Yang Fan <nullptr.cpp@gmail.com> | 2021-04-09 10:22:56 +0800 |
| commit | ca55f0511698e5b64538e02eff9f6b2280282b15 (patch) | |
| tree | ca5ef7a466c3fa8835619d741f1132f3ae5735b1 /clang/lib/Basic/SourceManager.cpp | |
| parent | 49c79e3b56448dbcd34aaf61e3678f7d6f5fcf80 (diff) | |
| download | llvm-ca55f0511698e5b64538e02eff9f6b2280282b15.zip llvm-ca55f0511698e5b64538e02eff9f6b2280282b15.tar.gz llvm-ca55f0511698e5b64538e02eff9f6b2280282b15.tar.bz2 | |
[clang][SourceManager] Fix -Wparentheses warning (NFC)
GCC warning:
```
/llvm-project/clang/lib/Basic/SourceManager.cpp: In instantiation of ‘constexpr T likelyhasbetween(T, unsigned char, unsigned char) [with T = long unsigned int]’:
/llvm-project/clang/lib/Basic/SourceManager.cpp:1292:52: required from here
/llvm-project/clang/lib/Basic/SourceManager.cpp:1264:48: warning: suggest parentheses around ‘+’ in operand of ‘&’ [-Wparentheses]
1264 | (x & ~static_cast<T>(0) / 255 * 127) +
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
1265 | (~static_cast<T>(0) / 255 * (127 - (m - 1)))) &
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
| -rw-r--r-- | clang/lib/Basic/SourceManager.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Basic/SourceManager.cpp b/clang/lib/Basic/SourceManager.cpp index 438525b..2c8036a 100644 --- a/clang/lib/Basic/SourceManager.cpp +++ b/clang/lib/Basic/SourceManager.cpp @@ -1261,8 +1261,8 @@ template <class T> static constexpr inline T likelyhasbetween(T x, unsigned char m, unsigned char n) { return ((x - ~static_cast<T>(0) / 255 * (n + 1)) & ~x & - (x & ~static_cast<T>(0) / 255 * 127) + - (~static_cast<T>(0) / 255 * (127 - (m - 1)))) & + ((x & ~static_cast<T>(0) / 255 * 127) + + (~static_cast<T>(0) / 255 * (127 - (m - 1))))) & ~static_cast<T>(0) / 255 * 128; } |