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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to