================
@@ -30,6 +30,9 @@ namespace doc {
 // SHA1'd hash of a USR.
 using SymbolID = std::array<uint8_t, 20>;
 
+static const SymbolID GlobalNamespace = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
----------------
ilovepi wrote:

```suggestion
constexpr SymbolID GlobalNamespaceID = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
```
I could imagine we may want to use GlobalNamespace as a name/concept.

Alternatively we could just have a helper API `isGlobalNamespace()` that does 
this check w/o needing a global constant. 

Also, I don't recall what putting `static` on a constant does in a header, but 
its probably better if it was just  use `constexpr` instead.

https://github.com/llvm/llvm-project/pull/162885
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to