From 8c33da5dc4144d6df474dcea986ca05932e2fe10 Mon Sep 17 00:00:00 2001 From: Gordon Henriksen Date: Mon, 17 Sep 2007 20:30:04 +0000 Subject: Fix for PR1633: Verifier doesn't fully verify GC intrinsics LLVM now enforces the following prototypes for the write barriers: * @llvm.gcread(*, **) void @llvm.gcwrite(*, *, **) And for @llvm.gcroot, the first stack slot is verified to be an alloca or a bitcast of an alloca. Fixes test/CodeGen/Generic/GC/lower_gcroot.ll, which violated these. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@42051 91177308-0d34-0410-b5e6-96231b3b80d8 --- test/Verifier/gcread-ptrptr.ll | 13 +++++++++++++ test/Verifier/gcroot-alloca.ll | 14 ++++++++++++++ test/Verifier/gcroot-meta.ll | 16 ++++++++++++++++ test/Verifier/gcroot-ptrptr.ll | 14 ++++++++++++++ test/Verifier/gcwrite-ptrptr.ll | 13 +++++++++++++ 5 files changed, 70 insertions(+) create mode 100644 test/Verifier/gcread-ptrptr.ll create mode 100644 test/Verifier/gcroot-alloca.ll create mode 100644 test/Verifier/gcroot-meta.ll create mode 100644 test/Verifier/gcroot-ptrptr.ll create mode 100644 test/Verifier/gcwrite-ptrptr.ll (limited to 'test/Verifier') diff --git a/test/Verifier/gcread-ptrptr.ll b/test/Verifier/gcread-ptrptr.ll new file mode 100644 index 0000000..bcfc6eb --- /dev/null +++ b/test/Verifier/gcread-ptrptr.ll @@ -0,0 +1,13 @@ +; RUN: not llvm-as < %s +; PR1633 + +%meta = type { i8* } +%obj = type { %meta* } + +declare %obj* @llvm.gcread(%obj*, %obj*) + +define %obj* @f() { +entry: + %x = call %obj* @llvm.gcread(%obj* null, %obj* null) + ret %obj* %x +} diff --git a/test/Verifier/gcroot-alloca.ll b/test/Verifier/gcroot-alloca.ll new file mode 100644 index 0000000..c44321a --- /dev/null +++ b/test/Verifier/gcroot-alloca.ll @@ -0,0 +1,14 @@ +; RUN: not llvm-as < %s +; PR1633 + +%meta = type { i8* } +%obj = type { %meta* } + +declare void @llvm.gcroot(%obj**, %meta*) + +define void @f() { +entry: + call void @llvm.gcroot(%obj** null, %meta* null) + + ret void +} diff --git a/test/Verifier/gcroot-meta.ll b/test/Verifier/gcroot-meta.ll new file mode 100644 index 0000000..51d1951 --- /dev/null +++ b/test/Verifier/gcroot-meta.ll @@ -0,0 +1,16 @@ +; RUN: not llvm-as < %s +; PR1633 + +%meta = type { i8* } +%obj = type { %meta* } + +declare void @llvm.gcroot(%obj**, %meta*) + +define void @f() { +entry: + %local.obj = alloca %obj* + %local.meta = alloca %meta + call void @llvm.gcroot(%obj** %local.obj, %meta* %local.meta) + + ret void +} diff --git a/test/Verifier/gcroot-ptrptr.ll b/test/Verifier/gcroot-ptrptr.ll new file mode 100644 index 0000000..0a7738d --- /dev/null +++ b/test/Verifier/gcroot-ptrptr.ll @@ -0,0 +1,14 @@ +; RUN: not llvm-as < %s +; PR1633 + +%meta = type { i8* } +%obj = type { %meta* } + +declare void @llvm.gcroot(%obj*, %meta*) + +define void @f() { +entry: + %local.obj = alloca %obj + call void @llvm.gcroot(%obj* %local.obj, %meta* null) + ret void +} diff --git a/test/Verifier/gcwrite-ptrptr.ll b/test/Verifier/gcwrite-ptrptr.ll new file mode 100644 index 0000000..b1e96c8 --- /dev/null +++ b/test/Verifier/gcwrite-ptrptr.ll @@ -0,0 +1,13 @@ +; RUN: not llvm-as < %s +; PR1633 + +%meta = type { i8* } +%obj = type { %meta* } + +declare void @llvm.gcwrite(%obj*, %obj*, %obj*) + +define void @f() { +entry: + call void @llvm.gcwrite(%obj* null, %obj* null, %obj* null) + ret void +} -- cgit v1.1