[flang-commits] [flang] [clang] [lld] [mlir] [llvm] [ELF] Merge exportDynamic into versionId (PR #71272)

[Alexander Richardson via flang-commits]

================
@@ -2518,9 +2521,9 @@ static void combineVersionedSymbol(Symbol &sym,
     sym.symbolKind = Symbol::PlaceholderKind;
     sym.isUsedInRegularObj = false;
   } else if (auto *sym1 = dyn_cast<Defined>(&sym)) {
-    if (sym2->versionId > VER_NDX_GLOBAL
-            ? config->versionDefinitions[sym2->versionId].name == suffix1 + 1
-            : sym1->section == sym2->section && sym1->value == sym2->value) {
+    if (sym2->versionId == VER_NDX_GLOBAL || sym2->versionId == nonExported
----------------
arichardson wrote:

I think this would be easier to read with parens, the operator precedence is not necessarily obvious.

https://github.com/llvm/llvm-project/pull/71272