steakhal wrote:

> > @necto are you interested measuring this patch? Do you have unrolling 
> > enabled? Do you have prior experience with this?
> 
> Thanks for the ping! We have loop unrolling enabled, so this patch is a no-op 
> for us. I will not evaluate it.

This is great. So we know that it doesn't crash, or impose really bad analysis 
times and I also know we had a look at outliers so this should be all pointing 
that this is fine. I'll still think about this on the weekend, but it's a 
really good indication.
Thank you for sharing!

https://github.com/llvm/llvm-project/pull/178911
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to