[LLD][ELF] .ARM.exidx sentinel section writeTo() should use InputSectionDescriptions.

[peter.smith]

Summary

This change converts the writing of the .ARM.exidx sentinel section to use the InputSectionDescriptions instead of OutputSection::Sections this is in preparation for the retirement of OutputSection::Sections.

To use the InputSectionDescriptions we need a way of obtaining them from the OutputSection. I've chosen to add member function to extract a vector of the InputSectionDescriptions. For this particular use case it might be better to offer a more functional Script->forEachInputSection(OutSecName, lambda) interface. However some
other use cases such as processing Thunks may need to insert directly into InputSections so I've used the interface that exposes these.

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 due to D33496 needing inputSectionRanges() so we don't need to add it here as well. This patch now depends on D33496.

[ruiu]

ELF/SyntheticSections.cpp

2190 ↗

(On Diff #100240)

This inputSectionRanges function doesn't seem to be defined. What am I missing?

[peter.smith]

The inputSectionRanges() was previously defined in D33496, I've removed it from that review to use Script->getCmd() instead. Sorry for the confusion.

I've now put it back here using an OutputSection* and getCmd() instead of StringRef, I've also started the search for the InputSection that precedes the Sentinel from the end as the sections are sorted in ascending order.

[peter.smith]

Updated patch to account for changes in D33496 which meant that we can guarantee that both the Sentinel and the preceding InputSection are in the same InputSectionDescription. This permits a simpler implementation here.

lld/trunk/ELF/SyntheticSections.cpp

 ARMExidxSentinelSection::ARMExidxSentinelSection()
     : SyntheticSection(SHF_ALLOC | SHF_LINK_ORDER, SHT_ARM_EXIDX,
                        Config->Wordsize, ".ARM.exidx") {}
 
 // Write a terminating sentinel entry to the end of the .ARM.exidx table.
 // This section will have been sorted last in the .ARM.exidx table.
 // This table entry will have the form:
 // | PREL31 upper bound of code that has exception tables | EXIDX_CANTUNWIND |
+// The sentinel must have the PREL31 value of an address higher than any
+// address described by any other table entry.
 void ARMExidxSentinelSection::writeTo(uint8_t *Buf) {
-  // Get the InputSection before us, we are by definition last
-  auto RI = this->OutSec->Sections.rbegin();
-  InputSection *LE = *(++RI);
-  InputSection *LC = cast<InputSection>(LE->getLinkOrderDep());
-  uint64_t S = LC->OutSec->Addr + LC->getOffset(LC->getSize());
-  uint64_t P = this->getVA();
+  // The Sections are sorted in order of ascending PREL31 address with the
+  // sentinel last. We need to find the InputSection that precedes the
+  // sentinel. By construction the Sentinel is in the last
+  // InputSectionDescription as the InputSection that precedes it.
+  OutputSectionCommand *C = Script->getCmd(OutSec);
+  auto ISD = std::find_if(C->Commands.rbegin(), C->Commands.rend(),
+                          [](const BaseCommand *Base) {
+                            return isa<InputSectionDescription>(Base);
+                          });
+  auto L = cast<InputSectionDescription>(*ISD);
+  InputSection *Highest = L->Sections[L->Sections.size() - 2];
+  InputSection *LS = cast<InputSection>(Highest->getLinkOrderDep());
+  uint64_t S = LS->OutSec->Addr + LS->getOffset(LS->getSize());
+  uint64_t P = getVA();
   Target->relocateOne(Buf, R_ARM_PREL31, S - P);
   write32le(Buf + 4, 0x1);
 }
 
 ThunkSection::ThunkSection(OutputSection *OS, uint64_t Off)
     : SyntheticSection(SHF_ALLOC | SHF_EXECINSTR, SHT_PROGBITS,
                        Config->Wordsize, ".text.thunk") {
   this->OutSec = OS;