diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2013-07-12 11:18:55 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2013-07-12 11:18:55 +0000 |
commit | cf3715caddf2b3cc2dfcae7bf79cec38d63dd78f (patch) | |
tree | f33511112d1db4c749fabc563bcf11d184efcd2f /clang/lib/Format/Format.cpp | |
parent | bcf1ca08e099a63c8df9d0a8ae568e44091ce8c8 (diff) | |
download | bcm5719-llvm-cf3715caddf2b3cc2dfcae7bf79cec38d63dd78f.tar.gz bcm5719-llvm-cf3715caddf2b3cc2dfcae7bf79cec38d63dd78f.zip |
Revert "indvars: Improve LFTR by eliminating truncation when comparing
against a constant."
This reverts commit r186107. It didn't handle wrapping arithmetic in the
loop correctly and thus caused the following C program to count from
0 to UINT64_MAX instead of from 0 to 255 as intended:
#include <stdio.h>
int main() {
unsigned char first = 0, last = 255;
do { printf("%d\n", first); } while (first++ != last);
}
Full test case and instructions to reproduce with just the -indvars pass
sent to the original review thread rather than to r186107's commit.
llvm-svn: 186152
Diffstat (limited to 'clang/lib/Format/Format.cpp')
0 files changed, 0 insertions, 0 deletions