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

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 7 11:09:25 PST 2023


================
@@ -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
----------------
MaskRay wrote:

I've seen some people use `(a && b) ? c : d`, but this isn't a majority.

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


More information about the cfe-commits mailing list