[clang] [Driver] Teach Barmetal toolchain about GCC installation (PR #121829)
Garvit Gupta via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 23 12:50:35 PDT 2025
quic-garvgupt wrote:
> I'm looking at this patch in isolation, so its possible that the comments here are addressed elasewhere.
>
> It looks like useful functionality to add to the bare metal driver.
>
> It also looks like there is some scope for some documentation, for users and for toolchain developers. Users will need to know about what options to set to point at their GCC installation (or place adjacent). Non-GNU toolchain developers will need to avoid `<triple>/lib/crt0.o`. If multilibs doesn't work for all toolchains yet, would be good to add that as a limitation.
>
> That could be done with a follow up patch though.
Thanks for providing feedback @smithp35. I can add documentation related to the points you mentioned. Pls let me know if adding them in the BareMetal.cpp file would be ok or should I consider adding it in some other file.
https://github.com/llvm/llvm-project/pull/121829
More information about the cfe-commits
mailing list