[LLD][ELF] Move creation of .ARM.exidx sentinel before LinkerScript creation

[peter.smith]

Summary

This change moves the creation of the .ARM.exidx sentinel section forward so that it is assigned to an OutputSection/InputSectionDescription with the existing .ARM.exidx sections.

A consequence of moving the .ARM.exidx creation earlier is that we can no longer guarantee that it will be already last in the table so we must account for this in the comparison function.

I've split this out from D33239 as it is a logical intermediate step towards converting the SHF_LINK_ORDER processing to use InputSectionDescriptions.

[peter.smith]

Updated patch to just append the .ARM.exidx sentinel section to an existing LinkerScript InputSectionDescription.

To do the insertion I've moved the LinkerScript::inputSectionRanges() function from D33500 so we can use it do the insertion and updated D33500.

[peter.smith]

Updated patch to use Script->getCmd(OS) as suggested.

I removed the comment "This function adds linker-created Out::* sections." as it seemed to be out of date as we don't have Out:: sections any more. I'm happy to put it back in or change it to something more specific like: " // Add any linker dependent sections that are dependent on the OutputSections"

ELF/Writer.cpp

if (InX::DynSymTab && S->includeInDynsym()) {
           In<ELFT>::VerNeed->addSymbol(SS);
     }
   }
 
   // Do not proceed if there was an undefined symbol.
   if (ErrorCount)
     return;
 
-  // So far we have added sections from input object files.
-  // This function adds linker-created Out::* sections.
   addPredefinedSections();
   removeUnusedSyntheticSections(OutputSections);
 
   sortSections();
 
   // This is a bit of a hack. A value of 0 means undef, so we set it
   // to 1 t make __ehdr_start defined. The section number is not
   // particularly relevant.
template <class ELFT> void Writer<ELFT>::finalizeSections() {
   applySynthetic({InX::SymTab, InX::ShStrTab, InX::StrTab},
                  [](SyntheticSection *SS) { SS->postThunkContents(); });
 }
 
 template <class ELFT> void Writer<ELFT>::addPredefinedSections() {
   // ARM ABI requires .ARM.exidx to be terminated by some piece of data.
   // We have the terminater synthetic section class. Add that at the end.
   auto *OS = dyn_cast_or_null<OutputSection>(findSection(".ARM.exidx"));
-  if (OS && !OS->Sections.empty() && !Config->Relocatable)
-    OS->addSection(make<ARMExidxSentinelSection>());
+  if (!OS || OS->Sections.empty() || Config->Relocatable)
+    return;
+
+  auto *Sentinel = make<ARMExidxSentinelSection>();
+  OS->addSection(Sentinel);
+  // If there are linker script commands existing at this point then add the
+  // sentinel to these too.
+  if (OutputSectionCommand *C = Script->getCmd(OS)) {
+    auto *ISD = make<InputSectionDescription>("");
+    ISD->Sections.push_back(Sentinel);
+  }
 }
 
 // The linker is expected to define SECNAME_start and SECNAME_end
 // symbols for a few sections. This function defines them.
 template <class ELFT> void Writer<ELFT>::addStartEndSymbols() {
   auto Define = [&](StringRef Start, StringRef End, OutputSection *OS) {
     // These symbols resolve to the image base if the section does not exist.
     // A special value -1 indicates end of the section.