| 97e50993c70083fdedb4f1dd2c487aa55c6f60cf |
|
29-Mar-2017 |
David Gross <dgross@google.com> |
Structure layout logic cleanup.
(Revert "Revert "Structure layout logic cleanup.""
This reverts commit 15440179aa78ae6ca1cc5207757691071a02b0bb.)
Do not run special x86-32 transformations when slang has explicitly
padded structs.
Verify that front end (Module) and back end (TargetMachine) agree on
the layout of every exported struct type.
Cannot build without slang change:
https://android-review.googlesource.com/#/c/359135/
Bug: http://b/29154200
Bug: http://b/28070272
Test: (aosp_x86_64-eng emulator, full_fugu-eng, aosp_angler-eng) x
(RsTest 32-bit, RsTest 64-bit, cts -m RenderscriptTest)
tests/run-lit-tests.sh
Tried (unmodified slang, modified bcc) and
( modified slang, unmodified bcc) and
( modified slang, modified bcc)
By instrumenting modified bcc, confimed that:
- Special x8632 layout transformations only run with unmodified slang,
and only when test is compiled for x8632.
"Modified slang" is a forthcoming slang change to add explicit padding
to struct types:
https://android-review.googlesource.com/#/c/359135/
Change-Id: Id938db661497710275a318e0964a36f64fcfa05b
/frameworks/compile/libbcc/lib/RSKernelExpand.cpp
|
| 15440179aa78ae6ca1cc5207757691071a02b0bb |
|
29-Mar-2017 |
David Gross <dgross@google.com> |
Revert "Structure layout logic cleanup."
This reverts commit 8281b5d72ff27077c461aa22e5c51a3a69e25916.
Change-Id: I3ed6617ca890c8dfd537b645357d0344746e549a
Reason: AddressSanitizer: container-overflow (aosp-master/aosp_x86_64-eng build)
/frameworks/compile/libbcc/lib/RSKernelExpand.cpp
|
| 8281b5d72ff27077c461aa22e5c51a3a69e25916 |
|
28-Mar-2017 |
David Gross <dgross@google.com> |
Structure layout logic cleanup.
Do not run special x86-32 transformations when slang has explicitly
padded structs.
Verify that front end (Module) and back end (TargetMachine) agree on
the layout of every exported struct type.
Cannot build without slang change:
https://android-review.googlesource.com/#/c/299366/
Bug: http://b/29154200
Bug: http://b/28070272
Test: (aosp_x86-eng emulator, full_fugu-eng, aosp_angler-eng) x
(RsTest 32-bit, RsTest 64-bit, cts -m RenderscriptTest)
tests/run-lit-tests.sh
Tried (unmodified slang, modified bcc) and
( modified slang, unmodified bcc) and
( modified slang, modified bcc)
By instrumenting modified bcc, confimed that:
- Special x8632 layout transformations only run with unmodified slang,
and only when test is compiled for x8632.
"Modified slang" is a forthcoming slang change to add explicit padding
to struct types:
https://android-review.googlesource.com/#/c/299366/
Change-Id: I536497d1152995bf93a48dc83527d1575d5f947e
/frameworks/compile/libbcc/lib/RSKernelExpand.cpp
|
| a2dd52f0710c214e00c1a13e25116e1af5eec77a |
|
17-Feb-2017 |
Jean-Luc Brouillet <jeanluc@google.com> |
Flatten the libbcc directories.
Get rid of the Support, Core, and RenderScript subdirectories for
the sources and the header files.
Deleted the unused Sha1 header file.
Bug: 34265954
Test: Compiled everything, ran a regular and a support lib app.
Change-Id: Iac760ee3c1985259593e7bbd1dd2dd35e99f654b
/frameworks/compile/libbcc/lib/RSKernelExpand.cpp
|