diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2011-04-06 17:35:32 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2011-04-06 17:35:32 +0000 |
commit | b4dd95b4f95c5ea954c834a9a8435222652f379d (patch) | |
tree | a032de623af8de1f8f5d9e667cf463c606e15439 /llvm/lib/Target/X86 | |
parent | 7a3b244d45f2ce7d059502438867ea9aa69da9a4 (diff) | |
download | bcm5719-llvm-b4dd95b4f95c5ea954c834a9a8435222652f379d.tar.gz bcm5719-llvm-b4dd95b4f95c5ea954c834a9a8435222652f379d.zip |
Add another case we are not optimizing.
llvm-svn: 129012
Diffstat (limited to 'llvm/lib/Target/X86')
-rw-r--r-- | llvm/lib/Target/X86/README.txt | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/README.txt b/llvm/lib/Target/X86/README.txt index b339e89e5a6..1ac2305e82b 100644 --- a/llvm/lib/Target/X86/README.txt +++ b/llvm/lib/Target/X86/README.txt @@ -1681,6 +1681,36 @@ Missed optimization: should be movl+andl. //===---------------------------------------------------------------------===// +The x86_64 abi says: + +Booleans, when stored in a memory object, are stored as single byte objects the +value of which is always 0 (false) or 1 (true). + +We are not using this fact: + +int bar(_Bool *a) { return *a; } + +define i32 @bar(i8* nocapture %a) nounwind readonly optsize { + %1 = load i8* %a, align 1, !tbaa !0 + %tmp = and i8 %1, 1 + %2 = zext i8 %tmp to i32 + ret i32 %2 +} + +bar: + movb (%rdi), %al + andb $1, %al + movzbl %al, %eax + ret + +GCC produces + +bar: + movzbl (%rdi), %eax + ret + +//===---------------------------------------------------------------------===// + Consider the following two functions compiled with clang: _Bool foo(int *x) { return !(*x & 4); } unsigned bar(int *x) { return !(*x & 4); } |