ilovepi wrote:

> Actually I'm going to go down the `Module::getCodeModel()` route. Right now I 
> think frontends adding module metadata for the code model is probably not 
> common, but we should aim for that. And this will in the short term still 
> work for Fuchsia without breaking API for other frontends. It just makes 
> updating tests more annoying since now they have to ingest IR with the proper 
> code model metadata.

I actually thought it was already available in the Module, and was surprised 
you had to pass TargetMachine around like that. If I understand where you're 
going with this, then I think that sounds like a really nice improvement.


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

Reply via email to