[clang] [RISCV] Merging RISCVToolChain and BareMetal toolchains (PR #118809)
Petr Hosek via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 12 00:58:26 PST 2024
petrhosek wrote:
> > I think this change could be made a lot smaller by doing a clean up first, bringing the BareMetal driver closer to other drivers, before attempting to merge the RISCVToolChain driver into it.
>
> Could you describe in a bit more detail which bits you think need to be cleaned up? Or are you planning to push additional changes yourself in the near future? This is a little vague. (We don't want this to be blocked indefinitely.)
Two areas I've specifically been looking into are `BareMetal::Linker::ConstructJob` and `BareMetal::AddClangCXXStdlibIncludeArgs`. I have local changes for both but they need more testing.
https://github.com/llvm/llvm-project/pull/118809
More information about the cfe-commits
mailing list