sihuan wrote:

> One thing you could do is open a PR with more testcases. As you point out, 
> the first case is covered by tests (though they need a TODO to say they 
> should be using a different instruction, if possible). For the second case, 
> adding `llc` tests along with a fixme that we need to get rid of the shifts 
> would help too. :)

I've opened a separate PR for the `llc` testcases here: #184070

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

Reply via email to