summaryrefslogtreecommitdiffstats
path: root/clang/lib/StaticAnalyzer/Checkers/IteratorChecker.cpp
diff options
context:
space:
mode:
authorPetr Pavlu <petr.pavlu@arm.com>2018-07-30 08:49:30 +0000
committerPetr Pavlu <petr.pavlu@arm.com>2018-07-30 08:49:30 +0000
commit8b6eff4e77f07f2afd738a97a821b5d292b9fdf1 (patch)
treec789ca421673f7e9f066fce464f039587c6c4d31 /clang/lib/StaticAnalyzer/Checkers/IteratorChecker.cpp
parent6a5c95bd6643a78b5bc1df7397444138956ffbc0 (diff)
downloadbcm5719-llvm-8b6eff4e77f07f2afd738a97a821b5d292b9fdf1.tar.gz
bcm5719-llvm-8b6eff4e77f07f2afd738a97a821b5d292b9fdf1.zip
[ARM] Fix over-alignment in arguments that are HA of 128-bit vectors
Code in `CC_ARM_AAPCS_Custom_Aggregate()` is responsible for handling homogeneous aggregates for `CC_ARM_AAPCS_VFP`. When an aggregate ends up fully on stack, the function tries to pack all resulting items of the aggregate as tightly as possible according to AAPCS. Once the first item was laid out, the alignment used for consecutive items was the size of one item. This logic went wrong for 128-bit vectors because their alignment is normally only 64 bits, and so could result in inserting unexpected padding between the first and second element. The patch fixes the problem by updating the alignment with the item size only if this results in reducing it. Differential Revision: https://reviews.llvm.org/D49720 llvm-svn: 338233
Diffstat (limited to 'clang/lib/StaticAnalyzer/Checkers/IteratorChecker.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud