diff options
| author | Paul Walker <paul.walker@arm.com> | 2019-08-16 17:29:53 +0000 |
|---|---|---|
| committer | Paul Walker <paul.walker@arm.com> | 2019-08-16 17:29:53 +0000 |
| commit | 2632c677f85cba1ac2aef5d68aaf8af0f5b3c944 (patch) | |
| tree | 9075736dd44e55c25bde501db4a1c9b5135f4d86 /clang/lib/Sema/SemaOpenMP.cpp | |
| parent | 19301d75f086caae1a495d267f5d0264b225942d (diff) | |
| download | bcm5719-llvm-2632c677f85cba1ac2aef5d68aaf8af0f5b3c944.tar.gz bcm5719-llvm-2632c677f85cba1ac2aef5d68aaf8af0f5b3c944.zip | |
[AArch64InstrInfo] Stop getInstSizeInBytes returning non-zero for meta instructions.
Recommit with fixes for mac builders.
Summary:
AArch64InstrInfo::getInstSizeInBytes is incorrectly treating meta
instructions (e.g. CFI_INSTRUCTION) as normal instructions and
giving them a size of 4.
This results in branch relaxation calculating block sizes wrong.
Branch relaxation also considers alignment and thus a single
mistake can result in later blocks being incorrectly sized even
when they themselves do not contain meta instructions.
The net result is we might not relax a branch whose destination is
not within range.
Reviewers: nickdesaulniers, peter.smith
Reviewed By: peter.smith
Subscribers: javed.absar, kristof.beyls, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D66337
> llvm-svn: 369111
llvm-svn: 369133
Diffstat (limited to 'clang/lib/Sema/SemaOpenMP.cpp')
0 files changed, 0 insertions, 0 deletions
