bjope wrote:

Here is another thing that I noticed after this patch: 
https://godbolt.org/z/1P7bnKGjh

So early instcombine is eliminating an `and` operation (in the foo example), 
resulting in simplifycfg not being able to collapse the control flow any longer.

Maybe I should file a separate issue for that?

Not sure if it should be consider as a phase ordering problem. Had perhaps been 
nice if simplifycfg had been able to understand that it basically could match 
an `and` with a no-op by using a -1 mask, etc.

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

Reply via email to