[PATCH] D77560: [SCEV] CreateNodeForPhi should return SCEVUnknown for incomplete PHIs

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 21:58:15 PDT 2020

Previous message: [PATCH] D79391: [DSE] Remove noop stores in MSSA.
Next message: [PATCH] D77560: [SCEV] CreateNodeForPhi should return SCEVUnknown for incomplete PHIs
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

mkazantsev added a comment.

I think we should just never query SCEVs for incomplete Phis as this operation makes no sense. It should be a bug in SCEV expander. Do you know why it makes this query to this Phi?


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

https://reviews.llvm.org/D77560

Previous message: [PATCH] D79391: [DSE] Remove noop stores in MSSA.
Next message: [PATCH] D77560: [SCEV] CreateNodeForPhi should return SCEVUnknown for incomplete PHIs
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list