diff options
| author | James Henderson <jh7370@my.bristol.ac.uk> | 2017-06-19 11:05:15 +0000 |
|---|---|---|
| committer | James Henderson <jh7370@my.bristol.ac.uk> | 2017-06-19 11:05:15 +0000 |
| commit | 38e67e82121f9e9fb8250117383f1d7917986040 (patch) | |
| tree | eb13cc2ef78311b341a1816bcc1499ee0616c077 /llvm/lib/Target/AArch64/AArch64MacroFusion.cpp | |
| parent | dad1e1a3b50b035eecfc5e225b736796a232b09c (diff) | |
| download | bcm5719-llvm-38e67e82121f9e9fb8250117383f1d7917986040.tar.gz bcm5719-llvm-38e67e82121f9e9fb8250117383f1d7917986040.zip | |
[ELF] Emit only one error if -z max-page-size without value
In r305364, Rui changed the mechanism that parses -z option values slightly.
This caused a bug, as demonstrated by this test, which now fails:
---
# REQUIRES: x86
# RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
# RUN: ld.lld %t.o -o %t -z max-page-size
.global _start
_start:
nop
---
Before, the link succeeded and set the max-page-size to the target default.
After we get the following two error messages:
"invalid max-page-size: "
"max-page-size: value isn't a power of 2"
The latter error is because an uninitialised variable ends up being passed back
to getMaxPageSize).
This change ensures we only get the first error.
Reviewers: ruiu
Differential Revision: https://reviews.llvm.org/D34234
llvm-svn: 305679
Diffstat (limited to 'llvm/lib/Target/AArch64/AArch64MacroFusion.cpp')
0 files changed, 0 insertions, 0 deletions
