xref: /freebsd/contrib/llvm-project/lld/ELF/ARMErrataFix.cpp (revision 2c2ec6bbc9cc7762a250ffe903bda6c2e44d25ff)
1 //===- ARMErrataFix.cpp ---------------------------------------------------===//
2 //
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4 // See https://llvm.org/LICENSE.txt for license information.
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6 //
7 //===----------------------------------------------------------------------===//
8 // This file implements Section Patching for the purpose of working around the
9 // Cortex-a8 erratum 657417 "A 32bit branch instruction that spans 2 4K regions
10 // can result in an incorrect instruction fetch or processor deadlock." The
11 // erratum affects all but r1p7, r2p5, r2p6, r3p1 and r3p2 revisions of the
12 // Cortex-A8. A high level description of the patching technique is given in
13 // the opening comment of AArch64ErrataFix.cpp.
14 //===----------------------------------------------------------------------===//
15 
16 #include "ARMErrataFix.h"
17 #include "InputFiles.h"
18 #include "LinkerScript.h"
19 #include "OutputSections.h"
20 #include "Relocations.h"
21 #include "Symbols.h"
22 #include "SyntheticSections.h"
23 #include "Target.h"
24 #include "llvm/Support/Endian.h"
25 #include <algorithm>
26 
27 using namespace llvm;
28 using namespace llvm::ELF;
29 using namespace llvm::object;
30 using namespace llvm::support;
31 using namespace llvm::support::endian;
32 using namespace lld;
33 using namespace lld::elf;
34 
35 // The documented title for Erratum 657417 is:
36 // "A 32bit branch instruction that spans two 4K regions can result in an
37 // incorrect instruction fetch or processor deadlock". Graphically using a
38 // 32-bit B.w instruction encoded as a pair of halfwords 0xf7fe 0xbfff
39 // xxxxxx000 // Memory region 1 start
40 // target:
41 // ...
42 // xxxxxxffe f7fe // First halfword of branch to target:
43 // xxxxxx000 // Memory region 2 start
44 // xxxxxx002 bfff // Second halfword of branch to target:
45 //
46 // The specific trigger conditions that can be detected at link time are:
47 // - There is a 32-bit Thumb-2 branch instruction with an address of the form
48 //   xxxxxxFFE. The first 2 bytes of the instruction are in 4KiB region 1, the
49 //   second 2 bytes are in region 2.
50 // - The branch instruction is one of BLX, BL, B.w BCC.w
51 // - The instruction preceding the branch is a 32-bit non-branch instruction.
52 // - The target of the branch is in region 1.
53 //
54 // The linker mitigation for the fix is to redirect any branch that meets the
55 // erratum conditions to a patch section containing a branch to the target.
56 //
57 // As adding patch sections may move branches onto region boundaries the patch
58 // must iterate until no more patches are added.
59 //
60 // Example, before:
61 // 00000FFA func: NOP.w      // 32-bit Thumb function
62 // 00000FFE       B.W func   // 32-bit branch spanning 2 regions, dest in 1st.
63 // Example, after:
64 // 00000FFA func: NOP.w      // 32-bit Thumb function
65 // 00000FFE       B.w __CortexA8657417_00000FFE
66 // 00001002       2 - bytes padding
67 // 00001004 __CortexA8657417_00000FFE: B.w func
68 
69 class elf::Patch657417Section final : public SyntheticSection {
70 public:
71   Patch657417Section(Ctx &, InputSection *p, uint64_t off, uint32_t instr,
72                      bool isARM);
73 
74   void writeTo(uint8_t *buf) override;
75 
76   size_t getSize() const override { return 4; }
77 
78   // Get the virtual address of the branch instruction at patcheeOffset.
79   uint64_t getBranchAddr() const;
80 
81   static bool classof(const SectionBase *d) {
82     return d->kind() == InputSectionBase::Synthetic && d->name ==".text.patch";
83   }
84 
85   // The Section we are patching.
86   const InputSection *patchee;
87   // The offset of the instruction in the Patchee section we are patching.
88   uint64_t patcheeOffset;
89   // A label for the start of the Patch that we can use as a relocation target.
90   Symbol *patchSym;
91   // A decoding of the branch instruction at patcheeOffset.
92   uint32_t instr;
93   // True If the patch is to be written in ARM state, otherwise the patch will
94   // be written in Thumb state.
95   bool isARM;
96 };
97 
98 // Return true if the half-word, when taken as the first of a pair of halfwords
99 // is the first half of a 32-bit instruction.
100 // Reference from ARM Architecture Reference Manual ARMv7-A and ARMv7-R edition
101 // section A6.3: 32-bit Thumb instruction encoding
102 // |             HW1                   |               HW2                |
103 // | 1 1 1 | op1 (2) | op2 (7) | x (4) |op|           x (15)              |
104 // With op1 == 0b00, a 16-bit instruction is encoded.
105 //
106 // We test only the first halfword, looking for op != 0b00.
107 static bool is32bitInstruction(uint16_t hw) {
108   return (hw & 0xe000) == 0xe000 && (hw & 0x1800) != 0x0000;
109 }
110 
111 // Reference from ARM Architecture Reference Manual ARMv7-A and ARMv7-R edition
112 // section A6.3.4 Branches and miscellaneous control.
113 // |             HW1              |               HW2                |
114 // | 1 1 1 | 1 0 | op (7) | x (4) | 1 | op1 (3) | op2 (4) | imm8 (8) |
115 // op1 == 0x0 op != x111xxx | Conditional branch (Bcc.W)
116 // op1 == 0x1               | Branch (B.W)
117 // op1 == 1x0               | Branch with Link and Exchange (BLX.w)
118 // op1 == 1x1               | Branch with Link (BL.W)
119 
120 static bool isBcc(uint32_t instr) {
121   return (instr & 0xf800d000) == 0xf0008000 &&
122          (instr & 0x03800000) != 0x03800000;
123 }
124 
125 static bool isB(uint32_t instr) { return (instr & 0xf800d000) == 0xf0009000; }
126 
127 static bool isBLX(uint32_t instr) { return (instr & 0xf800d000) == 0xf000c000; }
128 
129 static bool isBL(uint32_t instr) { return (instr & 0xf800d000) == 0xf000d000; }
130 
131 static bool is32bitBranch(uint32_t instr) {
132   return isBcc(instr) || isB(instr) || isBL(instr) || isBLX(instr);
133 }
134 
135 Patch657417Section::Patch657417Section(Ctx &ctx, InputSection *p, uint64_t off,
136                                        uint32_t instr, bool isARM)
137     : SyntheticSection(ctx, ".text.patch", SHT_PROGBITS,
138                        SHF_ALLOC | SHF_EXECINSTR, 4),
139       patchee(p), patcheeOffset(off), instr(instr), isARM(isARM) {
140   parent = p->getParent();
141   patchSym = addSyntheticLocal(
142       ctx, ctx.saver.save("__CortexA8657417_" + utohexstr(getBranchAddr())),
143       STT_FUNC, isARM ? 0 : 1, getSize(), *this);
144   addSyntheticLocal(ctx, ctx.saver.save(isARM ? "$a" : "$t"), STT_NOTYPE, 0, 0,
145                     *this);
146 }
147 
148 uint64_t Patch657417Section::getBranchAddr() const {
149   return patchee->getVA(patcheeOffset);
150 }
151 
152 // Given a branch instruction instr at sourceAddr work out its destination
153 // address. This is only used when the branch instruction has no relocation.
154 static uint64_t getThumbDestAddr(Ctx &ctx, uint64_t sourceAddr,
155                                  uint32_t instr) {
156   uint8_t buf[4];
157   write16le(buf, instr >> 16);
158   write16le(buf + 2, instr & 0x0000ffff);
159   int64_t offset;
160   if (isBcc(instr))
161     offset = ctx.target->getImplicitAddend(buf, R_ARM_THM_JUMP19);
162   else if (isB(instr))
163     offset = ctx.target->getImplicitAddend(buf, R_ARM_THM_JUMP24);
164   else
165     offset = ctx.target->getImplicitAddend(buf, R_ARM_THM_CALL);
166   // A BLX instruction from Thumb to Arm may have an address that is
167   // not 4-byte aligned. As Arm instructions are always 4-byte aligned
168   // the instruction is calculated (from Arm ARM):
169   // targetAddress = Align(PC, 4) + imm32
170   // where
171   //   Align(x, y) = y * (x Div y)
172   // which corresponds to alignDown.
173   if (isBLX(instr))
174     sourceAddr = alignDown(sourceAddr, 4);
175   return sourceAddr + offset + 4;
176 }
177 
178 void Patch657417Section::writeTo(uint8_t *buf) {
179   // The base instruction of the patch is always a 32-bit unconditional branch.
180   if (isARM)
181     write32le(buf, 0xea000000);
182   else
183     write32le(buf, 0x9000f000);
184   // If we have a relocation then apply it.
185   if (!relocs().empty()) {
186     ctx.target->relocateAlloc(*this, buf);
187     return;
188   }
189 
190   // If we don't have a relocation then we must calculate and write the offset
191   // ourselves.
192   // Get the destination offset from the addend in the branch instruction.
193   // We cannot use the instruction in the patchee section as this will have
194   // been altered to point to us!
195   uint64_t s = getThumbDestAddr(ctx, getBranchAddr(), instr);
196   // A BLX changes the state of the branch in the patch to Arm state, which
197   // has a PC Bias of 8, whereas in all other cases the branch is in Thumb
198   // state with a PC Bias of 4.
199   uint64_t pcBias = isBLX(instr) ? 8 : 4;
200   uint64_t p = getVA(pcBias);
201   ctx.target->relocateNoSym(buf, isARM ? R_ARM_JUMP24 : R_ARM_THM_JUMP24,
202                             s - p);
203 }
204 
205 // Given a branch instruction spanning two 4KiB regions, at offset off from the
206 // start of isec, return true if the destination of the branch is within the
207 // first of the two 4Kib regions.
208 static bool branchDestInFirstRegion(Ctx &ctx, const InputSection *isec,
209                                     uint64_t off, uint32_t instr,
210                                     const Relocation *r) {
211   uint64_t sourceAddr = isec->getVA(0) + off;
212   assert((sourceAddr & 0xfff) == 0xffe);
213   uint64_t destAddr;
214   // If there is a branch relocation at the same offset we must use this to
215   // find the destination address as the branch could be indirected via a thunk
216   // or the PLT.
217   if (r) {
218     uint64_t dst =
219         r->expr == R_PLT_PC ? r->sym->getPltVA(ctx) : r->sym->getVA(ctx);
220     // Account for Thumb PC bias, usually cancelled to 0 by addend of -4.
221     destAddr = dst + r->addend + 4;
222   } else {
223     // If there is no relocation, we must have an intra-section branch
224     // We must extract the offset from the addend manually.
225     destAddr = getThumbDestAddr(ctx, sourceAddr, instr);
226   }
227 
228   return (destAddr & 0xfffff000) == (sourceAddr & 0xfffff000);
229 }
230 
231 // Return true if a branch can reach a patch section placed after isec.
232 // The Bcc.w instruction has a range of 1 MiB, all others have 16 MiB.
233 static bool patchInRange(Ctx &ctx, const InputSection *isec, uint64_t off,
234                          uint32_t instr) {
235 
236   // We need the branch at source to reach a patch section placed immediately
237   // after isec. As there can be more than one patch in the patch section we
238   // add 0x100 as contingency to account for worst case of 1 branch every 4KiB
239   // for a 1 MiB range.
240   return ctx.target->inBranchRange(
241       isBcc(instr) ? R_ARM_THM_JUMP19 : R_ARM_THM_JUMP24, isec->getVA(off),
242       isec->getVA() + isec->getSize() + 0x100);
243 }
244 
245 struct ScanResult {
246   // Offset of branch within its InputSection.
247   uint64_t off;
248   // Cached decoding of the branch instruction.
249   uint32_t instr;
250   // Branch relocation at off. Will be nullptr if no relocation exists.
251   Relocation *rel;
252 };
253 
254 // Detect the erratum sequence, returning the offset of the branch instruction
255 // and a decoding of the branch. If the erratum sequence is not found then
256 // return an offset of 0 for the branch. 0 is a safe value to use for no patch
257 // as there must be at least one 32-bit non-branch instruction before the
258 // branch so the minimum offset for a patch is 4.
259 static ScanResult scanCortexA8Errata657417(InputSection *isec, uint64_t &off,
260                                            uint64_t limit) {
261   Ctx &ctx = isec->getCtx();
262   uint64_t isecAddr = isec->getVA(0);
263   // Advance Off so that (isecAddr + off) modulo 0x1000 is at least 0xffa. We
264   // need to check for a 32-bit instruction immediately before a 32-bit branch
265   // at 0xffe modulo 0x1000.
266   off = alignTo(isecAddr + off, 0x1000, 0xffa) - isecAddr;
267   if (off >= limit || limit - off < 8) {
268     // Need at least 2 4-byte sized instructions to trigger erratum.
269     off = limit;
270     return {0, 0, nullptr};
271   }
272 
273   ScanResult scanRes = {0, 0, nullptr};
274   const uint8_t *buf = isec->content().begin();
275   // ARMv7-A Thumb 32-bit instructions are encoded 2 consecutive
276   // little-endian halfwords.
277   const ulittle16_t *instBuf = reinterpret_cast<const ulittle16_t *>(buf + off);
278   uint16_t hw11 = *instBuf++;
279   uint16_t hw12 = *instBuf++;
280   uint16_t hw21 = *instBuf++;
281   uint16_t hw22 = *instBuf++;
282   if (is32bitInstruction(hw11) && is32bitInstruction(hw21)) {
283     uint32_t instr1 = (hw11 << 16) | hw12;
284     uint32_t instr2 = (hw21 << 16) | hw22;
285     if (!is32bitBranch(instr1) && is32bitBranch(instr2)) {
286       // Find a relocation for the branch if it exists. This will be used
287       // to determine the target.
288       uint64_t branchOff = off + 4;
289       auto relIt = llvm::find_if(isec->relocs(), [=](const Relocation &r) {
290         return r.offset == branchOff &&
291                (r.type == R_ARM_THM_JUMP19 || r.type == R_ARM_THM_JUMP24 ||
292                 r.type == R_ARM_THM_CALL);
293       });
294       if (relIt != isec->relocs().end())
295         scanRes.rel = &(*relIt);
296       if (branchDestInFirstRegion(ctx, isec, branchOff, instr2, scanRes.rel)) {
297         if (patchInRange(ctx, isec, branchOff, instr2)) {
298           scanRes.off = branchOff;
299           scanRes.instr = instr2;
300         } else {
301           Warn(ctx) << isec->file
302                     << ": skipping cortex-a8 657417 erratum sequence, section "
303                     << isec->name << " is too large to patch";
304         }
305       }
306     }
307   }
308   off += 0x1000;
309   return scanRes;
310 }
311 
312 void ARMErr657417Patcher::init() {
313   // The Arm ABI permits a mix of ARM, Thumb and Data in the same
314   // InputSection. We must only scan Thumb instructions to avoid false
315   // matches. We use the mapping symbols in the InputObjects to identify this
316   // data, caching the results in sectionMap so we don't have to recalculate
317   // it each pass.
318 
319   // The ABI Section 4.5.5 Mapping symbols; defines local symbols that describe
320   // half open intervals [Symbol Value, Next Symbol Value) of code and data
321   // within sections. If there is no next symbol then the half open interval is
322   // [Symbol Value, End of section). The type, code or data, is determined by
323   // the mapping symbol name, $a for Arm code, $t for Thumb code, $d for data.
324   auto isArmMapSymbol = [](const Symbol *s) {
325     return s->getName() == "$a" || s->getName().starts_with("$a.");
326   };
327   auto isThumbMapSymbol = [](const Symbol *s) {
328     return s->getName() == "$t" || s->getName().starts_with("$t.");
329   };
330   auto isDataMapSymbol = [](const Symbol *s) {
331     return s->getName() == "$d" || s->getName().starts_with("$d.");
332   };
333 
334   // Collect mapping symbols for every executable InputSection.
335   for (ELFFileBase *file : ctx.objectFiles) {
336     for (Symbol *s : file->getLocalSymbols()) {
337       auto *def = dyn_cast<Defined>(s);
338       if (!def)
339         continue;
340       if (!isArmMapSymbol(def) && !isThumbMapSymbol(def) &&
341           !isDataMapSymbol(def))
342         continue;
343       if (auto *sec = dyn_cast_or_null<InputSection>(def->section))
344         if (sec->flags & SHF_EXECINSTR)
345           sectionMap[sec].push_back(def);
346     }
347   }
348   // For each InputSection make sure the mapping symbols are in sorted in
349   // ascending order and are in alternating Thumb, non-Thumb order.
350   for (auto &kv : sectionMap) {
351     std::vector<const Defined *> &mapSyms = kv.second;
352     llvm::stable_sort(mapSyms, [](const Defined *a, const Defined *b) {
353       return a->value < b->value;
354     });
355     mapSyms.erase(llvm::unique(mapSyms,
356                                [=](const Defined *a, const Defined *b) {
357                                  return (isThumbMapSymbol(a) ==
358                                          isThumbMapSymbol(b));
359                                }),
360                   mapSyms.end());
361     // Always start with a Thumb Mapping Symbol
362     if (!mapSyms.empty() && !isThumbMapSymbol(mapSyms.front()))
363       mapSyms.erase(mapSyms.begin());
364   }
365   initialized = true;
366 }
367 
368 void ARMErr657417Patcher::insertPatches(
369     InputSectionDescription &isd, std::vector<Patch657417Section *> &patches) {
370   uint64_t spacing = 0x100000 - 0x7500;
371   uint64_t isecLimit;
372   uint64_t prevIsecLimit = isd.sections.front()->outSecOff;
373   uint64_t patchUpperBound = prevIsecLimit + spacing;
374   uint64_t outSecAddr = isd.sections.front()->getParent()->addr;
375 
376   // Set the outSecOff of patches to the place where we want to insert them.
377   // We use a similar strategy to initial thunk placement, using 1 MiB as the
378   // range of the Thumb-2 conditional branch with a contingency accounting for
379   // thunk generation.
380   auto patchIt = patches.begin();
381   auto patchEnd = patches.end();
382   for (const InputSection *isec : isd.sections) {
383     isecLimit = isec->outSecOff + isec->getSize();
384     if (isecLimit > patchUpperBound) {
385       for (; patchIt != patchEnd; ++patchIt) {
386         if ((*patchIt)->getBranchAddr() - outSecAddr >= prevIsecLimit)
387           break;
388         (*patchIt)->outSecOff = prevIsecLimit;
389       }
390       patchUpperBound = prevIsecLimit + spacing;
391     }
392     prevIsecLimit = isecLimit;
393   }
394   for (; patchIt != patchEnd; ++patchIt)
395     (*patchIt)->outSecOff = isecLimit;
396 
397   // Merge all patch sections. We use the outSecOff assigned above to
398   // determine the insertion point. This is ok as we only merge into an
399   // InputSectionDescription once per pass, and at the end of the pass
400   // assignAddresses() will recalculate all the outSecOff values.
401   SmallVector<InputSection *, 0> tmp;
402   tmp.reserve(isd.sections.size() + patches.size());
403   auto mergeCmp = [](const InputSection *a, const InputSection *b) {
404     if (a->outSecOff != b->outSecOff)
405       return a->outSecOff < b->outSecOff;
406     return isa<Patch657417Section>(a) && !isa<Patch657417Section>(b);
407   };
408   std::merge(isd.sections.begin(), isd.sections.end(), patches.begin(),
409              patches.end(), std::back_inserter(tmp), mergeCmp);
410   isd.sections = std::move(tmp);
411 }
412 
413 // Given a branch instruction described by ScanRes redirect it to a patch
414 // section containing an unconditional branch instruction to the target.
415 // Ensure that this patch section is 4-byte aligned so that the branch cannot
416 // span two 4 KiB regions. Place the patch section so that it is always after
417 // isec so the branch we are patching always goes forwards.
418 static void implementPatch(ScanResult sr, InputSection *isec,
419                            std::vector<Patch657417Section *> &patches) {
420   Ctx &ctx = isec->getCtx();
421   Log(ctx) << "detected cortex-a8-657419 erratum sequence starting at " <<
422       utohexstr(isec->getVA(sr.off)) << " in unpatched output";
423   Patch657417Section *psec;
424   // We have two cases to deal with.
425   // Case 1. There is a relocation at patcheeOffset to a symbol. The
426   // unconditional branch in the patch must have a relocation so that any
427   // further redirection via the PLT or a Thunk happens as normal. At
428   // patcheeOffset we redirect the existing relocation to a Symbol defined at
429   // the start of the patch section.
430   //
431   // Case 2. There is no relocation at patcheeOffset. We are unlikely to have
432   // a symbol that we can use as a target for a relocation in the patch section.
433   // Luckily we know that the destination cannot be indirected via the PLT or
434   // a Thunk so we can just write the destination directly.
435   if (sr.rel) {
436     // Case 1. We have an existing relocation to redirect to patch and a
437     // Symbol target.
438 
439     // Create a branch relocation for the unconditional branch in the patch.
440     // This can be redirected via the PLT or Thunks.
441     RelType patchRelType = R_ARM_THM_JUMP24;
442     int64_t patchRelAddend = sr.rel->addend;
443     bool destIsARM = false;
444     if (isBL(sr.instr) || isBLX(sr.instr)) {
445       // The final target of the branch may be ARM or Thumb, if the target
446       // is ARM then we write the patch in ARM state to avoid a state change
447       // Thunk from the patch to the target.
448       uint64_t dstSymAddr = (sr.rel->expr == R_PLT_PC)
449                                 ? sr.rel->sym->getPltVA(ctx)
450                                 : sr.rel->sym->getVA(ctx);
451       destIsARM = (dstSymAddr & 1) == 0;
452     }
453     psec = make<Patch657417Section>(ctx, isec, sr.off, sr.instr, destIsARM);
454     if (destIsARM) {
455       // The patch will be in ARM state. Use an ARM relocation and account for
456       // the larger ARM PC-bias of 8 rather than Thumb's 4.
457       patchRelType = R_ARM_JUMP24;
458       patchRelAddend -= 4;
459     }
460     psec->addReloc(
461         Relocation{sr.rel->expr, patchRelType, 0, patchRelAddend, sr.rel->sym});
462     // Redirect the existing branch relocation to the patch.
463     sr.rel->expr = R_PC;
464     sr.rel->addend = -4;
465     sr.rel->sym = psec->patchSym;
466   } else {
467     // Case 2. We do not have a relocation to the patch. Add a relocation of the
468     // appropriate type to the patch at patcheeOffset.
469 
470     // The destination is ARM if we have a BLX.
471     psec =
472         make<Patch657417Section>(ctx, isec, sr.off, sr.instr, isBLX(sr.instr));
473     RelType type;
474     if (isBcc(sr.instr))
475       type = R_ARM_THM_JUMP19;
476     else if (isB(sr.instr))
477       type = R_ARM_THM_JUMP24;
478     else
479       type = R_ARM_THM_CALL;
480     isec->addReloc(Relocation{R_PC, type, sr.off, -4, psec->patchSym});
481   }
482   patches.push_back(psec);
483 }
484 
485 // Scan all the instructions in InputSectionDescription, for each instance of
486 // the erratum sequence create a Patch657417Section. We return the list of
487 // Patch657417Sections that need to be applied to the InputSectionDescription.
488 std::vector<Patch657417Section *>
489 ARMErr657417Patcher::patchInputSectionDescription(
490     InputSectionDescription &isd) {
491   std::vector<Patch657417Section *> patches;
492   for (InputSection *isec : isd.sections) {
493     // LLD doesn't use the erratum sequence in SyntheticSections.
494     if (isa<SyntheticSection>(isec))
495       continue;
496     // Use sectionMap to make sure we only scan Thumb code and not Arm or inline
497     // data. We have already sorted mapSyms in ascending order and removed
498     // consecutive mapping symbols of the same type. Our range of executable
499     // instructions to scan is therefore [thumbSym->value, nonThumbSym->value)
500     // or [thumbSym->value, section size).
501     std::vector<const Defined *> &mapSyms = sectionMap[isec];
502 
503     auto thumbSym = mapSyms.begin();
504     while (thumbSym != mapSyms.end()) {
505       auto nonThumbSym = std::next(thumbSym);
506       uint64_t off = (*thumbSym)->value;
507       uint64_t limit = nonThumbSym == mapSyms.end() ? isec->content().size()
508                                                     : (*nonThumbSym)->value;
509 
510       while (off < limit) {
511         ScanResult sr = scanCortexA8Errata657417(isec, off, limit);
512         if (sr.off)
513           implementPatch(sr, isec, patches);
514       }
515       if (nonThumbSym == mapSyms.end())
516         break;
517       thumbSym = std::next(nonThumbSym);
518     }
519   }
520   return patches;
521 }
522 
523 bool ARMErr657417Patcher::createFixes() {
524   if (!initialized)
525     init();
526 
527   bool addressesChanged = false;
528   for (OutputSection *os : ctx.outputSections) {
529     if (!(os->flags & SHF_ALLOC) || !(os->flags & SHF_EXECINSTR))
530       continue;
531     for (SectionCommand *cmd : os->commands)
532       if (auto *isd = dyn_cast<InputSectionDescription>(cmd)) {
533         std::vector<Patch657417Section *> patches =
534             patchInputSectionDescription(*isd);
535         if (!patches.empty()) {
536           insertPatches(*isd, patches);
537           addressesChanged = true;
538         }
539       }
540   }
541   return addressesChanged;
542 }
543