libs-code/tree-sitter.tree-sitter
1
0
Fork 0
mirror of https://github.com/tree-sitter/tree-sitter.git synced 2026-09-10 07:26:23 -04:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

fix(loader): account for nm/ld fix on newer powerpc linux toolchains

Browse source 
Previously a bug in linux powerpc linkers/nm caused function symbols to
be incorrectly reported in the data "D" section. Newer toolchains now
correctly report these symbols' sections as "T". Account for both to
maintain compatibility with older toolchains
This commit is contained in:
Will Lillis 2026-01-28 10:14:15 -05:00
parent 69c6ce8c17
commit 18e2fb1e78
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
crates/loader/src/loader.rs
View file

@ -1295,10 +1295,13 @@ impl Loader {
} else {
""
};
let section = if cfg!(all(target_arch = "powerpc64", target_os = "linux")) {
" D "
let section = " T ";
// Older ppc toolchains incorrectly report functions in the Data section. This bug has been
// fixed, but we still need to account for older systems.
let old_ppc_section = if cfg!(all(target_arch = "powerpc64", target_os = "linux")) {
Some(" D ")
} else {
" T "
None
};
let mut must_have = vec![
format!("{prefix}tree_sitter_{name}_external_scanner_create"),
@ -1317,7 +1320,7 @@ impl Loader {
if output.status.success() {
let mut found_non_static = false;
for line in String::from_utf8_lossy(&output.stdout).lines() {
if line.contains(section) {
if line.contains(section) || old_ppc_section.is_some_and(|s| line.contains(s)) {
if let Some(function_name) =
line.split_whitespace().collect::<Vec<_>>().get(2)
{