h-vetinari wrote:

I think the title of the commit ("Support load lazy specialization lazily") is 
hard to parse and somewhat redundant. Perhaps the following would be better 
(IIUC): "Support loading template specializations lazily"

Also "We should a specialization ` A<ShouldNotBeLoaded>` in `M.cppm` [...]" in 
the body of the commit message is hard to parse. Perhaps what was meant was "We 
have a specialization [...]"?

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

Reply via email to