#include "llvm/IR/LLVMContext.h"
#include "llvm/IR/Module.h"
#include "llvm/IR/UseListOrder.h"
+#include "llvm/IR/Verifier.h"
#include "llvm/IRReader/IRReader.h"
#include "llvm/Support/CommandLine.h"
#include "llvm/Support/Debug.h"
static cl::opt<unsigned>
NumShuffles("num-shuffles",
cl::desc("Number of times to shuffle and verify use-lists"),
- cl::init(5));
+ cl::init(1));
namespace {
std::unique_ptr<Module> TempFile::readAssembly(LLVMContext &Context) const {
DEBUG(dbgs() << " - read assembly\n");
SMDiagnostic Err;
- std::unique_ptr<Module> M(ParseAssemblyFile(Filename, Err, Context));
+ std::unique_ptr<Module> M = parseAssemblyFile(Filename, Err, Context);
if (!M.get())
DEBUG(dbgs() << "error: "; Err.print("verify-use-list-order", dbgs()));
return M;
return true;
}
-static bool verifyBitcodeUseListOrder(const Module &M) {
- DEBUG(dbgs() << "*** verify-use-list-order: bitcode ***\n");
+static void verifyAfterRoundTrip(const Module &M,
+ std::unique_ptr<Module> OtherM) {
+ if (!OtherM)
+ report_fatal_error("parsing failed");
+ if (verifyModule(*OtherM, &errs()))
+ report_fatal_error("verification failed");
+ if (!matches(ValueMapping(M), ValueMapping(*OtherM)))
+ report_fatal_error("use-list order changed");
+}
+static void verifyBitcodeUseListOrder(const Module &M) {
+ errs() << "*** verify-use-list-order: bitcode ***\n";
TempFile F;
if (F.init("bc"))
- return false;
+ report_fatal_error("failed to initialize bitcode file");
if (F.writeBitcode(M))
- return false;
+ report_fatal_error("failed to write bitcode");
LLVMContext Context;
- std::unique_ptr<Module> OtherM = F.readBitcode(Context);
- if (!OtherM)
- return false;
-
- return matches(ValueMapping(M), ValueMapping(*OtherM));
+ verifyAfterRoundTrip(M, F.readBitcode(Context));
}
-static bool verifyAssemblyUseListOrder(const Module &M) {
- DEBUG(dbgs() << "*** verify-use-list-order: assembly ***\n");
+static void verifyAssemblyUseListOrder(const Module &M) {
+ errs() << "*** verify-use-list-order: assembly ***\n";
TempFile F;
if (F.init("ll"))
- return false;
+ report_fatal_error("failed to initialize assembly file");
if (F.writeAssembly(M))
- return false;
+ report_fatal_error("failed to write assembly");
LLVMContext Context;
- std::unique_ptr<Module> OtherM = F.readAssembly(Context);
- if (!OtherM)
- return false;
-
- return matches(ValueMapping(M), ValueMapping(*OtherM));
+ verifyAfterRoundTrip(M, F.readAssembly(Context));
}
static void verifyUseListOrder(const Module &M) {
- if (!verifyBitcodeUseListOrder(M))
- report_fatal_error("bitcode use-list order changed");
-
- if (shouldPreserveAssemblyUseListOrder())
- if (!verifyAssemblyUseListOrder(M))
- report_fatal_error("assembly use-list order changed");
+ verifyBitcodeUseListOrder(M);
+ verifyAssemblyUseListOrder(M);
}
static void shuffleValueUseLists(Value *V, std::minstd_rand0 &Gen,
});
}
-/// Shuffle all use-lists in a module.
-static void shuffleUseLists(Module &M, unsigned SeedOffset) {
- DEBUG(dbgs() << "*** shuffle-use-lists ***\n");
- std::minstd_rand0 Gen(std::minstd_rand0::default_seed + SeedOffset);
- DenseSet<Value *> Seen;
+static void reverseValueUseLists(Value *V, DenseSet<Value *> &Seen) {
+ if (!Seen.insert(V).second)
+ return;
- // Shuffle the use-list of each value that would be serialized to an IR file
- // (bitcode or assembly).
- auto shuffle = [&](Value *V) { shuffleValueUseLists(V, Gen, Seen); };
+ if (auto *C = dyn_cast<Constant>(V))
+ if (!isa<GlobalValue>(C))
+ for (Value *Op : C->operands())
+ reverseValueUseLists(Op, Seen);
+ if (V->use_empty() || std::next(V->use_begin()) == V->use_end())
+ // Nothing to shuffle for 0 or 1 users.
+ return;
+
+ DEBUG({
+ dbgs() << "V = ";
+ V->dump();
+ for (const Use &U : V->uses()) {
+ dbgs() << " - order: op = " << U.getOperandNo() << ", U = ";
+ U.getUser()->dump();
+ }
+ dbgs() << " => reverse\n";
+ });
+
+ V->reverseUseList();
+
+ DEBUG({
+ for (const Use &U : V->uses()) {
+ dbgs() << " - order: op = " << U.getOperandNo() << ", U = ";
+ U.getUser()->dump();
+ }
+ });
+}
+
+template <class Changer>
+static void changeUseLists(Module &M, Changer changeValueUseList) {
+ // Visit every value that would be serialized to an IR file.
+ //
// Globals.
for (GlobalVariable &G : M.globals())
- shuffle(&G);
+ changeValueUseList(&G);
for (GlobalAlias &A : M.aliases())
- shuffle(&A);
+ changeValueUseList(&A);
for (Function &F : M)
- shuffle(&F);
+ changeValueUseList(&F);
// Constants used by globals.
for (GlobalVariable &G : M.globals())
if (G.hasInitializer())
- shuffle(G.getInitializer());
+ changeValueUseList(G.getInitializer());
for (GlobalAlias &A : M.aliases())
- shuffle(A.getAliasee());
+ changeValueUseList(A.getAliasee());
for (Function &F : M)
if (F.hasPrefixData())
- shuffle(F.getPrefixData());
+ changeValueUseList(F.getPrefixData());
// Function bodies.
for (Function &F : M) {
for (Argument &A : F.args())
- shuffle(&A);
+ changeValueUseList(&A);
for (BasicBlock &BB : F)
- shuffle(&BB);
+ changeValueUseList(&BB);
for (BasicBlock &BB : F)
for (Instruction &I : BB)
- shuffle(&I);
+ changeValueUseList(&I);
// Constants used by instructions.
for (BasicBlock &BB : F)
for (Value *Op : I.operands())
if ((isa<Constant>(Op) && !isa<GlobalValue>(*Op)) ||
isa<InlineAsm>(Op))
- shuffle(Op);
+ changeValueUseList(Op);
}
+ if (verifyModule(M, &errs()))
+ report_fatal_error("verification failed");
+}
+
+static void shuffleUseLists(Module &M, unsigned SeedOffset) {
+ errs() << "*** shuffle-use-lists ***\n";
+ std::minstd_rand0 Gen(std::minstd_rand0::default_seed + SeedOffset);
+ DenseSet<Value *> Seen;
+ changeUseLists(M, [&](Value *V) { shuffleValueUseLists(V, Gen, Seen); });
+ DEBUG(dbgs() << "\n");
+}
+
+static void reverseUseLists(Module &M) {
+ errs() << "*** reverse-use-lists ***\n";
+ DenseSet<Value *> Seen;
+ changeUseLists(M, [&](Value *V) { reverseValueUseLists(V, Seen); });
DEBUG(dbgs() << "\n");
}
Err.print(argv[0], errs());
return 1;
}
+ if (verifyModule(*M, &errs()))
+ report_fatal_error("verification failed");
- DEBUG(dbgs() << "*** verify-use-list-order ***\n");
+ errs() << "*** verify-use-list-order ***\n";
+ // Can't verify if order isn't preserved.
if (!shouldPreserveBitcodeUseListOrder()) {
- // Can't verify if order isn't preserved.
- DEBUG(dbgs() << "warning: cannot verify bitcode; "
- "try -preserve-bc-use-list-order\n");
- return 0;
+ errs() << "warning: forcing -preserve-bc-use-list-order\n";
+ setPreserveBitcodeUseListOrder(true);
+ }
+ if (!shouldPreserveAssemblyUseListOrder()) {
+ errs() << "warning: forcing -preserve-ll-use-list-order\n";
+ setPreserveAssemblyUseListOrder(true);
}
+ // Verify the use lists now and after reversing them.
+ verifyUseListOrder(*M);
+ reverseUseLists(*M);
+ verifyUseListOrder(*M);
+
for (unsigned I = 0, E = NumShuffles; I != E; ++I) {
- DEBUG(dbgs() << "*** iteration: " << I << " ***\n");
+ errs() << "*** shuffle iteration: " << I + 1 << " of " << E << " ***\n";
- // Shuffle with a different seed each time so that use-lists that aren't
- // modified the first time are likely to be modified the next time.
+ // Shuffle with a different (deterministic) seed each time.
shuffleUseLists(*M, I);
+
+ // Verify again before and after reversing.
+ verifyUseListOrder(*M);
+ reverseUseLists(*M);
verifyUseListOrder(*M);
}