[PATCH] D122835: [SCEV] Fix a bug that caused an invalid assertion.

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 12:53:53 PDT 2022

Previous message: [PATCH] D122835: [SCEV] Fix a bug that caused an invalid assertion.
Next message: [PATCH] D124802: Custom IR printer for SLP that prints the cost after each vectorized instruction
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM, the simplified assert looks correct to me.  Removing complexity which is no longer needed is always good.


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

https://reviews.llvm.org/D122835

Previous message: [PATCH] D122835: [SCEV] Fix a bug that caused an invalid assertion.
Next message: [PATCH] D124802: Custom IR printer for SLP that prints the cost after each vectorized instruction
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list