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 | 6f0ec20e8fa23e9588205ea0eaba8f3021abe1ac (patch) | |
| tree | 4922e9ce6f2aed915948586459a599215edbc4b9 /lib/Object/COFFObjectFile.cpp | |
| parent | dd51a0c1e0b3cce8093244533b3505668d16f218 (diff) | |
| download | external_llvm-6f0ec20e8fa23e9588205ea0eaba8f3021abe1ac.zip external_llvm-6f0ec20e8fa23e9588205ea0eaba8f3021abe1ac.tar.gz external_llvm-6f0ec20e8fa23e9588205ea0eaba8f3021abe1ac.tar.bz2 | |
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.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186152 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Object/COFFObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions
