2013-11-30 19:39:27 +00:00
|
|
|
; RUN: llc -mcpu=ppc64 -O0 -fast-isel=false < %s | FileCheck %s
|
This patch addresses a PPC64 ELF issue with passing parameters consisting of
structs having size 3, 5, 6, or 7. Such a struct must be passed and received
as right-justified within its register or memory slot. The problem is only
present for structs that are passed in registers.
Previously, as part of a patch handling all structs of size less than 8, I
added logic to rotate the incoming register so that the struct was left-
justified prior to storing the whole register. This was incorrect because
the address of the parameter had already been adjusted earlier to point to
the right-adjusted value in the storage slot. Essentially I had accidentally
accounted for the right-adjustment twice.
In this patch, I removed the incorrect logic and reorganized the code to make
the flow clearer.
The removal of the rotates changes the expected code generation, so test case
structsinregs.ll has been modified to reflect this. I also added a new test
case, jaggedstructs.ll, to demonstrate that structs of these sizes can now
be properly received and passed.
I've built and tested the code on powerpc64-unknown-linux-gnu with no new
regressions. I also ran the GCC compatibility test suite and verified that
earlier problems with these structs are now resolved, with no new regressions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166680 91177308-0d34-0410-b5e6-96231b3b80d8
2012-10-25 13:38:09 +00:00
|
|
|
|
|
|
|
|
; This tests receiving and re-passing parameters consisting of structures
|
|
|
|
|
; of size 3, 5, 6, and 7. They are to be found/placed right-adjusted in
|
|
|
|
|
; the parameter registers.
|
|
|
|
|
|
|
|
|
|
target datalayout = "E-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v128:128:128-n32:64"
|
|
|
|
|
target triple = "powerpc64-unknown-linux-gnu"
|
|
|
|
|
|
|
|
|
|
%struct.S3 = type { [3 x i8] }
|
|
|
|
|
%struct.S5 = type { [5 x i8] }
|
|
|
|
|
%struct.S6 = type { [6 x i8] }
|
|
|
|
|
%struct.S7 = type { [7 x i8] }
|
|
|
|
|
|
|
|
|
|
define void @test(%struct.S3* byval %s3, %struct.S5* byval %s5, %struct.S6* byval %s6, %struct.S7* byval %s7) nounwind {
|
|
|
|
|
entry:
|
|
|
|
|
call void @check(%struct.S3* byval %s3, %struct.S5* byval %s5, %struct.S6* byval %s6, %struct.S7* byval %s7)
|
|
|
|
|
ret void
|
|
|
|
|
}
|
|
|
|
|
|
2013-06-13 20:23:34 +00:00
|
|
|
; CHECK: std 6, 184(1)
|
|
|
|
|
; CHECK: std 5, 176(1)
|
|
|
|
|
; CHECK: std 4, 168(1)
|
|
|
|
|
; CHECK: std 3, 160(1)
|
|
|
|
|
; CHECK: lbz {{[0-9]+}}, 167(1)
|
|
|
|
|
; CHECK: lhz {{[0-9]+}}, 165(1)
|
2013-03-27 02:40:14 +00:00
|
|
|
; CHECK: stb {{[0-9]+}}, 55(1)
|
This patch addresses a PPC64 ELF issue with passing parameters consisting of
structs having size 3, 5, 6, or 7. Such a struct must be passed and received
as right-justified within its register or memory slot. The problem is only
present for structs that are passed in registers.
Previously, as part of a patch handling all structs of size less than 8, I
added logic to rotate the incoming register so that the struct was left-
justified prior to storing the whole register. This was incorrect because
the address of the parameter had already been adjusted earlier to point to
the right-adjusted value in the storage slot. Essentially I had accidentally
accounted for the right-adjustment twice.
In this patch, I removed the incorrect logic and reorganized the code to make
the flow clearer.
The removal of the rotates changes the expected code generation, so test case
structsinregs.ll has been modified to reflect this. I also added a new test
case, jaggedstructs.ll, to demonstrate that structs of these sizes can now
be properly received and passed.
I've built and tested the code on powerpc64-unknown-linux-gnu with no new
regressions. I also ran the GCC compatibility test suite and verified that
earlier problems with these structs are now resolved, with no new regressions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166680 91177308-0d34-0410-b5e6-96231b3b80d8
2012-10-25 13:38:09 +00:00
|
|
|
; CHECK: sth {{[0-9]+}}, 53(1)
|
2013-06-13 20:23:34 +00:00
|
|
|
; CHECK: lbz {{[0-9]+}}, 175(1)
|
|
|
|
|
; CHECK: lwz {{[0-9]+}}, 171(1)
|
2013-03-27 02:40:14 +00:00
|
|
|
; CHECK: stb {{[0-9]+}}, 63(1)
|
This patch addresses a PPC64 ELF issue with passing parameters consisting of
structs having size 3, 5, 6, or 7. Such a struct must be passed and received
as right-justified within its register or memory slot. The problem is only
present for structs that are passed in registers.
Previously, as part of a patch handling all structs of size less than 8, I
added logic to rotate the incoming register so that the struct was left-
justified prior to storing the whole register. This was incorrect because
the address of the parameter had already been adjusted earlier to point to
the right-adjusted value in the storage slot. Essentially I had accidentally
accounted for the right-adjustment twice.
In this patch, I removed the incorrect logic and reorganized the code to make
the flow clearer.
The removal of the rotates changes the expected code generation, so test case
structsinregs.ll has been modified to reflect this. I also added a new test
case, jaggedstructs.ll, to demonstrate that structs of these sizes can now
be properly received and passed.
I've built and tested the code on powerpc64-unknown-linux-gnu with no new
regressions. I also ran the GCC compatibility test suite and verified that
earlier problems with these structs are now resolved, with no new regressions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166680 91177308-0d34-0410-b5e6-96231b3b80d8
2012-10-25 13:38:09 +00:00
|
|
|
; CHECK: stw {{[0-9]+}}, 59(1)
|
2013-06-13 20:23:34 +00:00
|
|
|
; CHECK: lhz {{[0-9]+}}, 182(1)
|
|
|
|
|
; CHECK: lwz {{[0-9]+}}, 178(1)
|
2013-03-27 02:40:14 +00:00
|
|
|
; CHECK: sth {{[0-9]+}}, 70(1)
|
This patch addresses a PPC64 ELF issue with passing parameters consisting of
structs having size 3, 5, 6, or 7. Such a struct must be passed and received
as right-justified within its register or memory slot. The problem is only
present for structs that are passed in registers.
Previously, as part of a patch handling all structs of size less than 8, I
added logic to rotate the incoming register so that the struct was left-
justified prior to storing the whole register. This was incorrect because
the address of the parameter had already been adjusted earlier to point to
the right-adjusted value in the storage slot. Essentially I had accidentally
accounted for the right-adjustment twice.
In this patch, I removed the incorrect logic and reorganized the code to make
the flow clearer.
The removal of the rotates changes the expected code generation, so test case
structsinregs.ll has been modified to reflect this. I also added a new test
case, jaggedstructs.ll, to demonstrate that structs of these sizes can now
be properly received and passed.
I've built and tested the code on powerpc64-unknown-linux-gnu with no new
regressions. I also ran the GCC compatibility test suite and verified that
earlier problems with these structs are now resolved, with no new regressions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166680 91177308-0d34-0410-b5e6-96231b3b80d8
2012-10-25 13:38:09 +00:00
|
|
|
; CHECK: stw {{[0-9]+}}, 66(1)
|
2013-06-13 20:23:34 +00:00
|
|
|
; CHECK: lbz {{[0-9]+}}, 191(1)
|
|
|
|
|
; CHECK: lhz {{[0-9]+}}, 189(1)
|
|
|
|
|
; CHECK: lwz {{[0-9]+}}, 185(1)
|
2013-03-27 02:40:14 +00:00
|
|
|
; CHECK: stb {{[0-9]+}}, 79(1)
|
|
|
|
|
; CHECK: sth {{[0-9]+}}, 77(1)
|
This patch addresses a PPC64 ELF issue with passing parameters consisting of
structs having size 3, 5, 6, or 7. Such a struct must be passed and received
as right-justified within its register or memory slot. The problem is only
present for structs that are passed in registers.
Previously, as part of a patch handling all structs of size less than 8, I
added logic to rotate the incoming register so that the struct was left-
justified prior to storing the whole register. This was incorrect because
the address of the parameter had already been adjusted earlier to point to
the right-adjusted value in the storage slot. Essentially I had accidentally
accounted for the right-adjustment twice.
In this patch, I removed the incorrect logic and reorganized the code to make
the flow clearer.
The removal of the rotates changes the expected code generation, so test case
structsinregs.ll has been modified to reflect this. I also added a new test
case, jaggedstructs.ll, to demonstrate that structs of these sizes can now
be properly received and passed.
I've built and tested the code on powerpc64-unknown-linux-gnu with no new
regressions. I also ran the GCC compatibility test suite and verified that
earlier problems with these structs are now resolved, with no new regressions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166680 91177308-0d34-0410-b5e6-96231b3b80d8
2012-10-25 13:38:09 +00:00
|
|
|
; CHECK: stw {{[0-9]+}}, 73(1)
|
|
|
|
|
; CHECK: ld 6, 72(1)
|
|
|
|
|
; CHECK: ld 5, 64(1)
|
|
|
|
|
; CHECK: ld 4, 56(1)
|
|
|
|
|
; CHECK: ld 3, 48(1)
|
|
|
|
|
|
|
|
|
|
declare void @check(%struct.S3* byval, %struct.S5* byval, %struct.S6* byval, %struct.S7* byval)
|
