[PATCH] D134020: [clang][Interp] Handle enums

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 23 13:20:24 PDT 2022


shafik accepted this revision.
shafik added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: clang/test/AST/Interp/enums.cpp:25
+  SIX = FIVE + 2,
+
+};
----------------
Maybe some edge case values for enumerators like `__INT_MAX__  *2U +1U` (UINT_MAX) 

and 

```
enum E { // warning: enumeration values exceed range of largest integer [-Wenum-too-large]
  E1 = -__LONG_MAX__ -1L,
  E2 = __LONG_MAX__ *2UL+1UL 
};
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134020/new/

https://reviews.llvm.org/D134020



More information about the cfe-commits mailing list