[PATCH] Incorrect loop limit computation in LFTR

David Majnemer david.majnemer at gmail.com
Fri Jan 23 02:14:02 PST 2015


Here was my thinking when I wrote the check you are modifying:

1. If we are going to compare against the post-incremented induction variable, the IndVar had better not have a wrapping flag.
2. Otherwise, we should compare against the pre-incremented induction variable.

Is there a reason why option number two isn't viable? I assumed that `IndVar` and `BackedgeTakenCount` would contain the correct values to compare.


http://reviews.llvm.org/D7137

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list