[PATCH] D17663: [JumpThreading] See through Cast Instructions

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 29 14:37:47 PST 2016


gberry added subscribers: sanjoy, majnemer.
gberry added a comment.

Making ComputeValueKnownInPredecessors() recursive for any cast instruction seems like it might be a bit heavy handed.  Perhaps you could limit it to casts of phi nodes as is done with CmpInsts farther down to reduce the compile time impact?

Perhaps someone more familiar with this pass (@majnemer, @sanjoy ?) could comment on this?


Repository:
  rL LLVM

http://reviews.llvm.org/D17663





More information about the llvm-commits mailing list