erichkeane wrote:

> I think this is fine the way it is, but I’ll leave it to @erichkeane to 
> approve this seeing as I proposed this approach here iirc.
> 
> Side note: please try to avoid force-pushing in the future as that makes it 
> really annoying to try and figure out what has changed since the last review. 
> We always squash on merge anyway, so a clean commit history on a pr is 
> irrelevant.

I'm away at LLVM dev right now. So won't get a chance to review until next 
week, but I promise this is one I'll look at asap

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

Reply via email to