Change memcpy/memset/memmove to have dest and source alignments.
This is a follow on from a similar LLVM commit: r253511. Note, this was reviewed (and more details are in) http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20151109/312083.html These intrinsics currently have an explicit alignment argument which is required to be a constant integer. It represents the alignment of the source and dest, and so must be the minimum of those. This change allows source and dest to each have their own alignments by using the alignment attribute on their arguments. The alignment argument itself is removed. The only code change to clang is hidden in CGBuilder.h which now passes both dest and source alignment to IRBuilder, instead of taking the minimum of dest and source alignments. Reviewed by Hal Finkel. llvm-svn: 253512
This commit is contained in:
@@ -23,47 +23,47 @@ struct foo {
|
||||
foo *test1(void *f, const foo &x) {
|
||||
return new (f) foo(x);
|
||||
// CHECK-POD: test1
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test1
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
}
|
||||
|
||||
foo *test2(const foo &x) {
|
||||
return new foo(x);
|
||||
// CHECK-POD: test2
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test2
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
}
|
||||
|
||||
foo test3(const foo &x) {
|
||||
foo f = x;
|
||||
return f;
|
||||
// CHECK-POD: test3
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test3
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
}
|
||||
|
||||
foo *test4(foo &&x) {
|
||||
return new foo(x);
|
||||
// CHECK-POD: test4
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test4
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
}
|
||||
|
||||
void test5(foo &f, const foo &x) {
|
||||
f = x;
|
||||
// CHECK-POD: test5
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test5
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 17, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 17, i1
|
||||
}
|
||||
|
||||
extern foo globtest;
|
||||
@@ -71,10 +71,10 @@ extern foo globtest;
|
||||
void test6(foo &&x) {
|
||||
globtest = move(x);
|
||||
// CHECK-POD: test6
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test6
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 17, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 17, i1
|
||||
}
|
||||
|
||||
void byval(foo f);
|
||||
@@ -82,8 +82,8 @@ void byval(foo f);
|
||||
void test7(const foo &x) {
|
||||
byval(x);
|
||||
// CHECK-POD: test7
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-POD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
|
||||
// CHECK-NONPOD: test7
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}}i64 24, i32 8
|
||||
// CHECK-NONPOD: call void @llvm.memcpy.p0i8.p0i8.i64({{.*}} align 8 {{.*}} align 8 {{.*}} i64 24, i1
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user