replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[analyzer] MoveChecker: Squash the bit field because it causes a GCC warning. 

The warning seems spurious (GCC bug 51242), but the bit field is
simply not worth the hassle.

rdar://problem/41349073

llvm-svn: 349394
This commit is contained in:
Artem Dergachev 2018-12-17 21:07:38 +00:00
parent 38cda981a2
commit b4bde2acee
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
clang/lib/StaticAnalyzer/Checkers/MoveChecker.cpp
View File

@ -63,9 +63,7 @@ public:
private:
enum MisuseKind { MK_FunCall, MK_Copy, MK_Move, MK_Dereference };
// This needs to be unsigned in order to avoid undefined behavior
// when putting it into a tight bitfield.
enum StdObjectKind : unsigned { SK_NonStd, SK_Unsafe, SK_Safe, SK_SmartPtr };
enum StdObjectKind { SK_NonStd, SK_Unsafe, SK_Safe, SK_SmartPtr };
enum AggressivenessKind { // In any case, don't warn after a reset.
AK_Invalid = -1,
@ -81,9 +79,9 @@ private:
struct ObjectKind {
// Is this a local variable or a local rvalue reference?
bool IsLocal : 1;
bool IsLocal;
// Is this an STL object? If so, of what kind?
StdObjectKind StdKind : 2;
StdObjectKind StdKind;
};
// STL smart pointers are automatically re-initialized to null when moved