momchil-velikov wrote:

> > > The combine itself is missed because of multiple splats, identical in 
> > > value, but different in SSA name.
> > 
> > 
> > Are we missing a run of CSE somewhere?
> 
> I think I can post a fix for this in VPlan CSE soon, it looks relatively 
> straightforward. We just need to teach it to handle single scalar loads. I 
> don't think it needs to block this PR.

I have a fix for it, just some polishing before submitting a PR.

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

Reply via email to