summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/ARM
diff options
context:
space:
mode:
authorOleg Ranevskyy <oranevskyy@accesssoftek.com>2017-05-23 19:38:37 +0000
committerOleg Ranevskyy <oranevskyy@accesssoftek.com>2017-05-23 19:38:37 +0000
commit09df0020fc572c8bfe3c99ec9b30a55cb3b8dc29 (patch)
tree98553893804dfdd52f4a59d0e1b6af3bc95b23ef /llvm/lib/Target/ARM
parent452c8aec61cf855e1c01f1fa334d6e77e9a48ac6 (diff)
downloadbcm5719-llvm-09df0020fc572c8bfe3c99ec9b30a55cb3b8dc29.tar.gz
bcm5719-llvm-09df0020fc572c8bfe3c99ec9b30a55cb3b8dc29.zip
[ARM] Temporarily disable globals promotion to constant pools to prevent miscompilation
Summary: A temporary workaround for PR32780 - rematerialized instructions accessing the same promoted global through different constant pool entries. The patch turns off the globals promotion optimization leaving all its code in place, so that it can be easily turned on once PR32780 is fixed. Since this is a miscompilation issue causing generation of misbehaving code, and the problem is very subtle, the patch might be valuable enough to get into 4.0.1. Reviewers: efriedma, jmolloy Reviewed By: efriedma Subscribers: aemerson, javed.absar, llvm-commits, rengolin, asl, tstellar Differential Revision: https://reviews.llvm.org/D33446 llvm-svn: 303679
Diffstat (limited to 'llvm/lib/Target/ARM')
-rw-r--r--llvm/lib/Target/ARM/ARMISelLowering.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp
index f8b584db7b9..4bfb9b01656 100644
--- a/llvm/lib/Target/ARM/ARMISelLowering.cpp
+++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp
@@ -127,7 +127,7 @@ static cl::opt<bool> EnableConstpoolPromotion(
"arm-promote-constant", cl::Hidden,
cl::desc("Enable / disable promotion of unnamed_addr constants into "
"constant pools"),
- cl::init(true));
+ cl::init(false)); // FIXME: set to true by default once PR32780 is fixed
static cl::opt<unsigned> ConstpoolPromotionMaxSize(
"arm-promote-constant-max-size", cl::Hidden,
cl::desc("Maximum size of constant to promote into a constant pool"),
OpenPOWER on IntegriCloud