[PATCH] D41003: Silence GCC 7 warning by using an enum class.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 10 07:58:58 PST 2018


RKSimon accepted this revision.
RKSimon added a comment.

LGTM



================
Comment at: lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp:95
 
+enum class ColorKind { None, Red, Black };
 
----------------
(style) Worth using a CK_ prefix?

```

enum class ColorKind { CK_None, CK_Red, CK_Black };
```


https://reviews.llvm.org/D41003





More information about the llvm-commits mailing list