| ; RUN: llc -mtriple=x86_64-apple-macosx -O3 -debug-only=faultmaps -enable-implicit-null-checks < %s 2>&1 | FileCheck %s |
| ; REQUIRES: asserts |
| |
| ; List cases where we should *not* be emitting implicit null checks. |
| |
| ; CHECK-NOT: Fault Map Output |
| |
| define i32 @imp_null_check_load(i32* %x, i32* %y) { |
| entry: |
| %c = icmp eq i32* %x, null |
| ; It isn't legal to move the load from %x from "not_null" to here -- |
| ; the store to %y could be aliasing it. |
| br i1 %c, label %is_null, label %not_null, !make.implicit !0 |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| store i32 0, i32* %y |
| %t = load i32, i32* %x |
| ret i32 %t |
| } |
| |
| define i32 @imp_null_check_gep_load(i32* %x) { |
| entry: |
| %c = icmp eq i32* %x, null |
| br i1 %c, label %is_null, label %not_null, !make.implicit !0 |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| ; null + 5000 * sizeof(i32) lies outside the null page and hence the |
| ; load to %t cannot be assumed to be reliably faulting. |
| %x.gep = getelementptr i32, i32* %x, i32 5000 |
| %t = load i32, i32* %x.gep |
| ret i32 %t |
| } |
| |
| define i32 @imp_null_check_neg_gep_load(i32* %x) { |
| entry: |
| %c = icmp eq i32* %x, null |
| br i1 %c, label %is_null, label %not_null, !make.implicit !0 |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| ; null - 5000 * sizeof(i32) lies outside the null page and hence the |
| ; load to %t cannot be assumed to be reliably faulting. |
| %x.gep = getelementptr i32, i32* %x, i32 -5000 |
| %t = load i32, i32* %x.gep |
| ret i32 %t |
| } |
| |
| define i32 @imp_null_check_load_no_md(i32* %x) { |
| ; This is fine, except it is missing the !make.implicit metadata. |
| entry: |
| %c = icmp eq i32* %x, null |
| br i1 %c, label %is_null, label %not_null |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| %t = load i32, i32* %x |
| ret i32 %t |
| } |
| |
| define i32 @imp_null_check_no_hoist_over_acquire_load(i32* %x, i32* %y) { |
| ; We cannot hoist %t1 over %t0 since %t0 is an acquire load |
| entry: |
| %c = icmp eq i32* %x, null |
| br i1 %c, label %is_null, label %not_null, !make.implicit !0 |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| %t0 = load atomic i32, i32* %y acquire, align 4 |
| %t1 = load i32, i32* %x |
| %p = add i32 %t0, %t1 |
| ret i32 %p |
| } |
| |
| define i32 @imp_null_check_add_result(i32* %x, i32* %y) { |
| ; This will codegen to: |
| ; |
| ; movl (%rsi), %eax |
| ; addl (%rdi), %eax |
| ; |
| ; The load instruction we wish to hoist is the addl, but there is a |
| ; write-after-write hazard preventing that from happening. We could |
| ; get fancy here and exploit the commutativity of addition, but right |
| ; now -implicit-null-checks isn't that smart. |
| ; |
| |
| entry: |
| %c = icmp eq i32* %x, null |
| br i1 %c, label %is_null, label %not_null, !make.implicit !0 |
| |
| is_null: |
| ret i32 42 |
| |
| not_null: |
| %t0 = load i32, i32* %y |
| %t1 = load i32, i32* %x |
| %p = add i32 %t0, %t1 |
| ret i32 %p |
| } |
| |
| !0 = !{} |