[Openmp-commits] [PATCH] D125717: [InstCombine] Optimize and of icmps with power-of-2 and contiguous masks

Noah Goldstein via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed May 31 15:38:45 PDT 2023


goldstein.w.n added a comment.

In D125717#4385553 <https://reviews.llvm.org/D125717#4385553>, @jmciver wrote:

> @nikic this patch is for #54856 <https://github.com/llvm/llvm-project/issues/54856>, which originated from @erikdesjardins.
>
> This refinment is not triggered by building LLVM or anything of the components of the llvm-test-suite.
>
> What are the appropriate next steps to gain InstCombine maintainer sign off?
>
> Thanks in advance for your feedback.

you might try @RKSimon.

Also to be fair I do end-to-end reviews (just not a maintainer) and believe this is okay to go out as is. If you don't hear back
soon I don't think it would be an issue to push.
I can run the zorg buildbot tests on this for you if it would help you feel more comfortable about it, or of course you can
wait as you see fit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125717/new/

https://reviews.llvm.org/D125717



More information about the Openmp-commits mailing list