aeubanks wrote:

> > > yes, but I'd like to that after we collect feedback from first users
> > 
> > 
> > They are introduced by earlier transformations
> > Note: I'd like to have special intrinsic for this optimization. When we 
> > have it, we likely don't need this SimplifyCFG.
> 
> lgtm with a comment added on why we're adding the extra SimplifyCFG

oh I missed that you'd added one. but maybe a TODO with your comment about an 
intrinsic?

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

Reply via email to