[PATCH] D146629: [SimplifyCFG][LICM] Preserve nonnull, range and align metadata when speculating

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 09:22:37 PDT 2023


jdoerfert added a comment.

In D146629#4213446 <https://reviews.llvm.org/D146629#4213446>, @nikic wrote:

> @jdoerfert Yeah, there is a pending patch that fixes this for `!range` here: https://reviews.llvm.org/D142687 We'll need similar changes for nonnull/align as well.
>
> I'll take this off the review queue until those changes have all landed.

Just a thought:
Can we create a test that we somehow run against all transformations (one by one) to verify they don't combine good metadata?


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

https://reviews.llvm.org/D146629



More information about the llvm-commits mailing list