https://github.com/smithp35 commented:

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.

https://github.com/llvm/llvm-project/pull/121829
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to