Merged main:ba976971898d74df38d155c55e008c898120d1e4 into amd-gfx:77eaf56bc521

Local branch amd-gfx 77eaf56 Merged main:4fe33d067c5d0894d0059418f09edc531f16ac9f into amd-gfx:5fa38fbc60f8
Remote branch main ba97697 [NVPTX] support switch statement with brx.idx (llvm#102400)

Author: SC llvm team

.github/CODEOWNERS

@@ -33,9 +33,6 @@
 /clang/www/cxx_dr_status.html @Endilll
 /clang/www/make_cxx_dr_status @Endilll
 
-clang/lib/AST/Interp/ @tbaederr
-clang/test/AST/Interp/ @tbaederr
-
 /clang/include/clang/CIR @lanza @bcardosolopes
 /clang/lib/CIR @lanza @bcardosolopes
 /clang/tools/cir-* @lanza @bcardosolopes

.github/workflows/libclang-python-tests.yml

@@ -22,12 +22,6 @@ on:
       - '.github/workflows/libclang-python-tests.yml'
       - '.github/workflows/llvm-project-tests.yml'
 
-concurrency:
-  # Skip intermediate builds: always.
-  # Cancel intermediate builds: only if it is a pull request build.
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: ${{ startsWith(github.ref, 'refs/pull/') }}
-
 jobs:
   check-clang-python:
     # Build libclang and then run the libclang Python binding's unit tests.

.github/workflows/llvm-project-tests.yml

@@ -51,7 +51,7 @@ concurrency:
   # Cancel intermediate builds: only if it is a pull request build.
   # If the group name here is the same as the group name in the workflow that includes
   # this one, then the action will try to wait on itself and get stuck.
-  group: llvm-project-${{ github.workflow }}-${{ inputs.projects }}${{ github.ref }}
+  group: llvm-project-${{ github.workflow }}-${{ inputs.projects }}-${{ inputs.python_version }}${{ github.ref }}
   cancel-in-progress: ${{ startsWith(github.ref, 'refs/pull/') }}
 
 jobs:

bolt/include/bolt/Core/BinaryContext.h

@@ -911,8 +911,7 @@ class BinaryContext {
   /// of \p Flags.
   MCSymbol *registerNameAtAddress(StringRef Name, uint64_t Address,
                                   uint64_t Size, uint16_t Alignment,
-                                  unsigned Flags = 0,
-                                  BinarySection *Section = NULL);
+                                  unsigned Flags = 0);
 
   /// Return BinaryData registered at a given \p Address or nullptr if no
   /// global symbol was registered at the location.

bolt/lib/Core/BinaryContext.cpp

@@ -1055,28 +1055,18 @@ void BinaryContext::adjustCodePadding() {
 MCSymbol *BinaryContext::registerNameAtAddress(StringRef Name, uint64_t Address,
                                                uint64_t Size,
                                                uint16_t Alignment,
-                                               unsigned Flags,
-                                               BinarySection *Section) {
+                                               unsigned Flags) {
   // Register the name with MCContext.
   MCSymbol *Symbol = Ctx->getOrCreateSymbol(Name);
-  BinaryData *BD;
-
-  // Register out of section symbols only in GlobalSymbols map
-  if (Section && Section->getEndAddress() == Address) {
-    BD = new BinaryData(*Symbol, Address, Size, Alignment ? Alignment : 1,
-                        *Section, Flags);
-    GlobalSymbols[Name] = BD;
-    return Symbol;
-  }
 
   auto GAI = BinaryDataMap.find(Address);
+  BinaryData *BD;
   if (GAI == BinaryDataMap.end()) {
     ErrorOr<BinarySection &> SectionOrErr = getSectionForAddress(Address);
-    BinarySection &SectionRef = Section        ? *Section
-                                : SectionOrErr ? SectionOrErr.get()
-                                               : absoluteSection();
+    BinarySection &Section =
+        SectionOrErr ? SectionOrErr.get() : absoluteSection();
     BD = new BinaryData(*Symbol, Address, Size, Alignment ? Alignment : 1,
-                        SectionRef, Flags);
+                        Section, Flags);
     GAI = BinaryDataMap.emplace(Address, BD).first;
     GlobalSymbols[Name] = BD;
     updateObjectNesting(GAI);
@@ -1411,7 +1401,7 @@ void BinaryContext::postProcessSymbolTable() {
     if ((BD->getName().starts_with("SYMBOLat") ||
          BD->getName().starts_with("DATAat")) &&
         !BD->getParent() && !BD->getSize() && !BD->isAbsolute() &&
-        BD->getSection().getSize()) {
+        BD->getSection()) {
       this->errs() << "BOLT-WARNING: zero-sized top level symbol: " << *BD
                    << "\n";
       Valid = false;

bolt/lib/Rewrite/RewriteInstance.cpp

@@ -956,13 +956,13 @@ void RewriteInstance::discoverFileObjects() {
     uint64_t SymbolSize = ELFSymbolRef(Symbol).getSize();
     uint64_t SymbolAlignment = Symbol.getAlignment();
 
-    auto registerName = [&](uint64_t FinalSize, BinarySection *Section = NULL) {
+    auto registerName = [&](uint64_t FinalSize) {
       // Register names even if it's not a function, e.g. for an entry point.
       BC->registerNameAtAddress(UniqueName, SymbolAddress, FinalSize,
-                                SymbolAlignment, SymbolFlags, Section);
+                                SymbolAlignment, SymbolFlags);
       if (!AlternativeName.empty())
         BC->registerNameAtAddress(AlternativeName, SymbolAddress, FinalSize,
-                                  SymbolAlignment, SymbolFlags, Section);
+                                  SymbolAlignment, SymbolFlags);
     };
 
     section_iterator Section =
@@ -987,25 +987,12 @@ void RewriteInstance::discoverFileObjects() {
                       << " for function\n");
 
     if (SymbolAddress == Section->getAddress() + Section->getSize()) {
-      ErrorOr<BinarySection &> SectionOrError =
-          BC->getSectionForAddress(Section->getAddress());
-
-      // Skip symbols from invalid sections
-      if (!SectionOrError) {
-        BC->errs() << "BOLT-WARNING: " << UniqueName << " (0x"
-                   << Twine::utohexstr(SymbolAddress)
-                   << ") does not have any section\n";
-        continue;
-      }
-
       assert(SymbolSize == 0 &&
              "unexpect non-zero sized symbol at end of section");
-      LLVM_DEBUG({
-        dbgs() << "BOLT-DEBUG: rejecting as symbol " << UniqueName
-               << " points to end of " << SectionOrError->getName()
-               << " section\n";
-      });
-      registerName(SymbolSize, &SectionOrError.get());
+      LLVM_DEBUG(
+          dbgs()
+          << "BOLT-DEBUG: rejecting as symbol points to end of its section\n");
+      registerName(SymbolSize);
       continue;
     }
 
@@ -2637,30 +2624,6 @@ void RewriteInstance::handleRelocation(const SectionRef &RelocatedSection,
     }
   }
 
-  if (Relocation::isGOT(RType) && !Relocation::isTLS(RType)) {
-    auto exitOnGotEndSymol = [&](StringRef Name) {
-      BC->errs() << "BOLT-ERROR: GOT table contains currently unsupported "
-                    "section end symbol "
-                 << Name << "\n";
-      exit(1);
-    };
-
-    if (SymbolIter != InputFile->symbol_end() && ReferencedSection) {
-      if (cantFail(SymbolIter->getAddress()) ==
-          ReferencedSection->getEndAddress())
-        exitOnGotEndSymol(cantFail(SymbolIter->getName()));
-    } else {
-      // If no section and symbol are provided by relocation, try to find the
-      // symbol by its name, including the possibility that the symbol is local.
-      BinaryData *BD = BC->getBinaryDataByName(SymbolName);
-      if (!BD && NR.getUniquifiedNameCount(SymbolName) == 1)
-        BD = BC->getBinaryDataByName(NR.getUniqueName(SymbolName, 1));
-
-      if ((BD && BD->getAddress() == BD->getSection().getEndAddress()))
-        exitOnGotEndSymol(BD->getName());
-    }
-  }
-
   if (!ReferencedSection)
     ReferencedSection = BC->getSectionForAddress(SymbolAddress);
 

bolt/test/AArch64/Inputs/got_end_of_section_symbol.lld_script

@@ -1,15 +1,15 @@
 ## Check that BOLT doesn't consider end-of-section symbols (e.g., _etext) as
 ## functions.
 
-# REQUIRES: system-linux, asserts
+# REQUIRES: x86_64-linux, asserts
 
 # RUN: llvm-mc -filetype=obj -triple x86_64-unknown-linux %s -o %t.o
 # RUN: ld.lld %t.o -o %t.exe -q
 # RUN: llvm-bolt %t.exe -o %t.null --print-cfg --debug-only=bolt 2>&1 \
 # RUN:   | FileCheck %s
 
 # CHECK: considering symbol etext for function
-# CHECK-NEXT: rejecting as symbol etext points to end of .text section
+# CHECK-NEXT: rejecting as symbol points to end of its section
 # CHECK-NOT: Binary Function "etext{{.*}}" after building cfg
 
 

bolt/test/AArch64/got_end_of_section_symbol.s

@@ -384,5 +384,12 @@ ClangDocContext::ClangDocContext(tooling::ExecutionContext *ECtx,
   }
 }
 
+void ScopeChildren::sort() {
+  llvm::sort(Namespaces.begin(), Namespaces.end());
+  llvm::sort(Records.begin(), Records.end());
+  llvm::sort(Functions.begin(), Functions.end());
+  llvm::sort(Enums.begin(), Enums.end());
+  llvm::sort(Typedefs.begin(), Typedefs.end());
+}
 } // namespace doc
 } // namespace clang