Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CBE doesn't handle signlessness right #1458

Closed
lattner opened this issue Jan 7, 2007 · 2 comments
Closed

CBE doesn't handle signlessness right #1458

lattner opened this issue Jan 7, 2007 · 2 comments
Labels
bugzilla Issues migrated from bugzilla miscompilation

Comments

@lattner
Copy link
Collaborator

lattner commented Jan 7, 2007

Bugzilla Link 1086
Resolution FIXED
Resolved on Feb 22, 2010 12:47
Version trunk
OS All

Extended Description

Consider:

int test(unsigned short X, short Y) { return X+Y; }

$ llvm-gcc t.c -c -emit-llvm -o - -O3 | llc -march=c
signed int test(signed short ltmp_0_1, signed short ltmp_1_1) {
return ((((signed int )(signed short )ltmp_1_1)) + (((unsigned int )(unsigned short )ltmp_0_1)));
}

The first argument should be unsigned. This is a serious bug, because it means that bugpoint will get
confused more often than it currently does.

-Chris

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 7, 2007

Fixed. No llvm-commits email so no patch :(

Reid.

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 7, 2007

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla miscompilation
Projects
None yet
Development

No branches or pull requests

2 participants