From: Mehdi Amini <mehdi.amini@apple.com>
Date: Wed, 14 Dec 2016 02:35:32 +0000 (+0000)
Subject: Don't double-initialize cl::opt for iterating in reverse order to uncover non-determi... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5e5a55a4a088d625c64238a91dd86e3046760845;p=llvm

Don't double-initialize cl::opt for iterating in reverse order to uncover non-determinism in codegen by default

Bots are broken and needs to be fixed before having this on by default.
The feature was committed in r289619.

I tried to disable it in r289624 and failed because it was initialized in two places.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@289626 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Support/CommandLine.cpp b/lib/Support/CommandLine.cpp
index f4d501aceba..fa1782c9941 100644
--- a/lib/Support/CommandLine.cpp
+++ b/lib/Support/CommandLine.cpp
@@ -52,7 +52,7 @@ namespace llvm {
 // This will help uncover differences in codegen caused due to undefined
 // iteration order.
 static cl::opt<bool, true> ReverseIteration("reverse-iterate",
-  cl::location(ReverseIterate<bool>::value), cl::init(true));
+  cl::location(ReverseIterate<bool>::value));
 }
 #endif