Skip to content

Conversation

@takusuman
Copy link
Contributor

…M 1st stage

Discovered this while building Copacabana's toolchain. Fixes #72.

…M 1st stage

Discovered this while building Copacabana's toolchain.
Fixes dslm4515#72.
@dslm4515
Copy link
Owner

dslm4515 commented Feb 9, 2025

For the 2nd line edit, I think you forgot to use $TARGET_TUPLE instead of a architecture-specific one

@takusuman
Copy link
Contributor Author

For the 2nd line edit, I think you forgot to use $TARGET_TUPLE instead of a architecture-specific one

Sure, just one minute.

@takusuman
Copy link
Contributor Author

Well, I think this also happens in other parts of the file, take a look:

# Should output:
# ld.lld: /llvmtools/lib/Scrt1.o
# ld.lld: /llvmtools/lib/crti.o
# ld.lld: /mnt/cmlfs/cgnutools/lib/clang/17/lib/x86_64-pc-linux-musl/clang_rt.crtbegin.o
# ld.lld: /tmp/dummy-ffb57c.o
# ld.lld: /mnt/cmlfs/cgnutools/lib/clang/17/lib/x86_64-pc-linux-musl/libclang_rt.builtins.a
# ld.lld: /llvmtools/lib/libunwind.so
# ld.lld: /llvmtools/lib/libatomic.so
# ld.lld: /llvmtools/lib/libc.so
# ld.lld: /mnt/cmlfs/cgnutools/lib/clang/17/lib/x86_64-pc-linux-musl/libclang_rt.builtins.a
# ld.lld: /llvmtools/lib/libunwind.so
# ld.lld: /mnt/cmlfs/cgnutools/lib/clang/17/lib/x86_64-pc-linux-musl/clang_rt.crtend.o
# ld.lld: /llvmtools/lib/crtn.o

That's why I used an architecture-specific example, and I think we could also assume that someone who's reading CMLFS knows that there are parts of the expected lld --- or pretty much any other verification --- output that will be architecture specific.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TO-DO: Libc++'s __config_site misplaced

2 participants