1*9880d681SAndroid Build Coastguard Worker; RUN: llc -mtriple=x86_64-apple-macosx -O3 -debug-only=faultmaps -enable-implicit-null-checks < %s 2>&1 | FileCheck %s 2*9880d681SAndroid Build Coastguard Worker; REQUIRES: asserts 3*9880d681SAndroid Build Coastguard Worker 4*9880d681SAndroid Build Coastguard Worker; List cases where we should *not* be emitting implicit null checks. 5*9880d681SAndroid Build Coastguard Worker 6*9880d681SAndroid Build Coastguard Worker; CHECK-NOT: Fault Map Output 7*9880d681SAndroid Build Coastguard Worker 8*9880d681SAndroid Build Coastguard Workerdefine i32 @imp_null_check_load(i32* %x, i32* %y) { 9*9880d681SAndroid Build Coastguard Worker entry: 10*9880d681SAndroid Build Coastguard Worker %c = icmp eq i32* %x, null 11*9880d681SAndroid Build Coastguard Worker; It isn't legal to move the load from %x from "not_null" to here -- 12*9880d681SAndroid Build Coastguard Worker; the store to %y could be aliasing it. 13*9880d681SAndroid Build Coastguard Worker br i1 %c, label %is_null, label %not_null, !make.implicit !0 14*9880d681SAndroid Build Coastguard Worker 15*9880d681SAndroid Build Coastguard Worker is_null: 16*9880d681SAndroid Build Coastguard Worker ret i32 42 17*9880d681SAndroid Build Coastguard Worker 18*9880d681SAndroid Build Coastguard Worker not_null: 19*9880d681SAndroid Build Coastguard Worker store i32 0, i32* %y 20*9880d681SAndroid Build Coastguard Worker %t = load i32, i32* %x 21*9880d681SAndroid Build Coastguard Worker ret i32 %t 22*9880d681SAndroid Build Coastguard Worker} 23*9880d681SAndroid Build Coastguard Worker 24*9880d681SAndroid Build Coastguard Workerdefine i32 @imp_null_check_gep_load(i32* %x) { 25*9880d681SAndroid Build Coastguard Worker entry: 26*9880d681SAndroid Build Coastguard Worker %c = icmp eq i32* %x, null 27*9880d681SAndroid Build Coastguard Worker br i1 %c, label %is_null, label %not_null, !make.implicit !0 28*9880d681SAndroid Build Coastguard Worker 29*9880d681SAndroid Build Coastguard Worker is_null: 30*9880d681SAndroid Build Coastguard Worker ret i32 42 31*9880d681SAndroid Build Coastguard Worker 32*9880d681SAndroid Build Coastguard Worker not_null: 33*9880d681SAndroid Build Coastguard Worker; null + 5000 * sizeof(i32) lies outside the null page and hence the 34*9880d681SAndroid Build Coastguard Worker; load to %t cannot be assumed to be reliably faulting. 35*9880d681SAndroid Build Coastguard Worker %x.gep = getelementptr i32, i32* %x, i32 5000 36*9880d681SAndroid Build Coastguard Worker %t = load i32, i32* %x.gep 37*9880d681SAndroid Build Coastguard Worker ret i32 %t 38*9880d681SAndroid Build Coastguard Worker} 39*9880d681SAndroid Build Coastguard Worker 40*9880d681SAndroid Build Coastguard Workerdefine i32 @imp_null_check_load_no_md(i32* %x) { 41*9880d681SAndroid Build Coastguard Worker; This is fine, except it is missing the !make.implicit metadata. 42*9880d681SAndroid Build Coastguard Worker entry: 43*9880d681SAndroid Build Coastguard Worker %c = icmp eq i32* %x, null 44*9880d681SAndroid Build Coastguard Worker br i1 %c, label %is_null, label %not_null 45*9880d681SAndroid Build Coastguard Worker 46*9880d681SAndroid Build Coastguard Worker is_null: 47*9880d681SAndroid Build Coastguard Worker ret i32 42 48*9880d681SAndroid Build Coastguard Worker 49*9880d681SAndroid Build Coastguard Worker not_null: 50*9880d681SAndroid Build Coastguard Worker %t = load i32, i32* %x 51*9880d681SAndroid Build Coastguard Worker ret i32 %t 52*9880d681SAndroid Build Coastguard Worker} 53*9880d681SAndroid Build Coastguard Worker 54*9880d681SAndroid Build Coastguard Workerdefine i32 @imp_null_check_no_hoist_over_acquire_load(i32* %x, i32* %y) { 55*9880d681SAndroid Build Coastguard Worker; We cannot hoist %t1 over %t0 since %t0 is an acquire load 56*9880d681SAndroid Build Coastguard Worker entry: 57*9880d681SAndroid Build Coastguard Worker %c = icmp eq i32* %x, null 58*9880d681SAndroid Build Coastguard Worker br i1 %c, label %is_null, label %not_null, !make.implicit !0 59*9880d681SAndroid Build Coastguard Worker 60*9880d681SAndroid Build Coastguard Worker is_null: 61*9880d681SAndroid Build Coastguard Worker ret i32 42 62*9880d681SAndroid Build Coastguard Worker 63*9880d681SAndroid Build Coastguard Worker not_null: 64*9880d681SAndroid Build Coastguard Worker %t0 = load atomic i32, i32* %y acquire, align 4 65*9880d681SAndroid Build Coastguard Worker %t1 = load i32, i32* %x 66*9880d681SAndroid Build Coastguard Worker %p = add i32 %t0, %t1 67*9880d681SAndroid Build Coastguard Worker ret i32 %p 68*9880d681SAndroid Build Coastguard Worker} 69*9880d681SAndroid Build Coastguard Worker 70*9880d681SAndroid Build Coastguard Workerdefine i32 @imp_null_check_add_result(i32* %x, i32* %y) { 71*9880d681SAndroid Build Coastguard Worker; This will codegen to: 72*9880d681SAndroid Build Coastguard Worker; 73*9880d681SAndroid Build Coastguard Worker; movl (%rsi), %eax 74*9880d681SAndroid Build Coastguard Worker; addl (%rdi), %eax 75*9880d681SAndroid Build Coastguard Worker; 76*9880d681SAndroid Build Coastguard Worker; The load instruction we wish to hoist is the addl, but there is a 77*9880d681SAndroid Build Coastguard Worker; write-after-write hazard preventing that from happening. We could 78*9880d681SAndroid Build Coastguard Worker; get fancy here and exploit the commutativity of addition, but right 79*9880d681SAndroid Build Coastguard Worker; now -implicit-null-checks isn't that smart. 80*9880d681SAndroid Build Coastguard Worker; 81*9880d681SAndroid Build Coastguard Worker 82*9880d681SAndroid Build Coastguard Worker entry: 83*9880d681SAndroid Build Coastguard Worker %c = icmp eq i32* %x, null 84*9880d681SAndroid Build Coastguard Worker br i1 %c, label %is_null, label %not_null, !make.implicit !0 85*9880d681SAndroid Build Coastguard Worker 86*9880d681SAndroid Build Coastguard Worker is_null: 87*9880d681SAndroid Build Coastguard Worker ret i32 42 88*9880d681SAndroid Build Coastguard Worker 89*9880d681SAndroid Build Coastguard Worker not_null: 90*9880d681SAndroid Build Coastguard Worker %t0 = load i32, i32* %y 91*9880d681SAndroid Build Coastguard Worker %t1 = load i32, i32* %x 92*9880d681SAndroid Build Coastguard Worker %p = add i32 %t0, %t1 93*9880d681SAndroid Build Coastguard Worker ret i32 %p 94*9880d681SAndroid Build Coastguard Worker} 95*9880d681SAndroid Build Coastguard Worker 96*9880d681SAndroid Build Coastguard Worker!0 = !{} 97