================
@@ -115,15 +115,17 @@ static int initialize(Lang Language) {
       NSLen = 0;
     }
 
-    if (SymIndex >= 0 &&
-        Mapping->SymbolNames[SymIndex].qualifiedName() == QName) {
-      // Not a new symbol, use the same index.
+    if (SymIndex > 0) {
       assert(llvm::none_of(llvm::ArrayRef(Mapping->SymbolNames, SymIndex),
----------------
vvd170501 wrote:

This assertion could fail only if entries for a symbol were partially grouped, 
but it didn't fail if entries weren't grouped at all.
Example:
```cpp
// StdSpecialSymbolMap.inc
SYMBOL(any_cast, std::, <any>)
SYMBOL(div, std::, <cstdlib>)
// StdSymbolMap.inc
SYMBOL(any, std::, <any>)
SYMBOL(any_cast, std::, <any>)  // assertion doesn't fail because previous 
entry has a different name
...

```

https://github.com/llvm/llvm-project/pull/113612
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to