gamesh411 wrote:

> What's the relationship between this PR and #93799 ?
> 
> Otherwise, the change LGTM, but you might want to either unify these two NFC 
> changes into a single commit or ensure that they're independent.

Thanks for checking this!
To be honest, I started working on this, and then it hit me that a test case 
for the limitation would be nice 😸 
My intention was to ease the review as much as possible by cutting them into 
atomic parts.
They could indeed be independent; I will leave the limitation out of this and 
just introduce it in the other patch.


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

Reply via email to