[lld] [lld] Error on unsupported split stack (PR #88063)

via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 8 15:56:18 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lld-elf

Author: Patrick O'Neill (patrick-rivos)

<details>
<summary>Changes</summary>

Targets with no `-fstack-split` support now emit `ld.lld: error: Target doesn't support split stacks.` instead of `UNREACHABLE executed` with a backtrace asking the user to report a bug.

Resolves #<!-- -->88061

---
Full diff: https://github.com/llvm/llvm-project/pull/88063.diff


1 Files Affected:

- (modified) lld/ELF/Target.cpp (+1-1) 


``````````diff
diff --git a/lld/ELF/Target.cpp b/lld/ELF/Target.cpp
index d879a427e9c096..735b2a82d851b4 100644
--- a/lld/ELF/Target.cpp
+++ b/lld/ELF/Target.cpp
@@ -141,7 +141,7 @@ bool TargetInfo::needsThunk(RelExpr expr, RelType type, const InputFile *file,
 
 bool TargetInfo::adjustPrologueForCrossSplitStack(uint8_t *loc, uint8_t *end,
                                                   uint8_t stOther) const {
-  llvm_unreachable("Target doesn't support split stacks.");
+  fatal("Target doesn't support split stacks.");
 }
 
 bool TargetInfo::inBranchRange(RelType type, uint64_t src, uint64_t dst) const {

``````````

</details>


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


More information about the llvm-commits mailing list