Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(500)

Unified Diff: lib/Transforms/NaCl/ExpandConstantExpr.cpp

Issue 939073008: Rebased PNaCl localmods in LLVM to 223109 (Closed)
Patch Set: undo localmod Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/Transforms/NaCl/ExpandByVal.cpp ('k') | lib/Transforms/NaCl/ExpandCtors.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/Transforms/NaCl/ExpandConstantExpr.cpp
diff --git a/lib/Transforms/NaCl/ExpandConstantExpr.cpp b/lib/Transforms/NaCl/ExpandConstantExpr.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..3c3b317b7d91c7e803a8a56efcdb8a66c9712344
--- /dev/null
+++ b/lib/Transforms/NaCl/ExpandConstantExpr.cpp
@@ -0,0 +1,94 @@
+//===- ExpandConstantExpr.cpp - Convert ConstantExprs to Instructions------===//
+//
+// The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+//
+// This pass expands out ConstantExprs into Instructions.
+//
+// Note that this only converts ConstantExprs that are referenced by
+// Instructions. It does not convert ConstantExprs that are used as
+// initializers for global variables.
+//
+// This simplifies the language so that the PNaCl translator does not
+// need to handle ConstantExprs as part of a stable wire format for
+// PNaCl.
+//
+//===----------------------------------------------------------------------===//
+
+#include <map>
+
+#include "llvm/IR/Constants.h"
+#include "llvm/IR/Function.h"
+#include "llvm/IR/Instructions.h"
+#include "llvm/Pass.h"
+#include "llvm/Transforms/NaCl.h"
+
+using namespace llvm;
+
+static bool expandInstruction(Instruction *Inst);
+
+namespace {
+ // This is a FunctionPass because our handling of PHI nodes means
+ // that our modifications may cross BasicBlocks.
+ struct ExpandConstantExpr : public FunctionPass {
+ static char ID; // Pass identification, replacement for typeid
+ ExpandConstantExpr() : FunctionPass(ID) {
+ initializeExpandConstantExprPass(*PassRegistry::getPassRegistry());
+ }
+
+ virtual bool runOnFunction(Function &Func);
+ };
+}
+
+char ExpandConstantExpr::ID = 0;
+INITIALIZE_PASS(ExpandConstantExpr, "expand-constant-expr",
+ "Expand out ConstantExprs into Instructions",
+ false, false)
+
+static Value *expandConstantExpr(Instruction *InsertPt, ConstantExpr *Expr) {
+ Instruction *NewInst = Expr->getAsInstruction();
+ NewInst->insertBefore(InsertPt);
+ NewInst->setName("expanded");
+ expandInstruction(NewInst);
+ return NewInst;
+}
+
+static bool expandInstruction(Instruction *Inst) {
+ // A landingpad can only accept ConstantExprs, so it should remain
+ // unmodified.
+ if (isa<LandingPadInst>(Inst))
+ return false;
+
+ bool Modified = false;
+ for (unsigned OpNum = 0; OpNum < Inst->getNumOperands(); OpNum++) {
+ if (ConstantExpr *Expr =
+ dyn_cast<ConstantExpr>(Inst->getOperand(OpNum))) {
+ Modified = true;
+ Use *U = &Inst->getOperandUse(OpNum);
+ PhiSafeReplaceUses(U, expandConstantExpr(PhiSafeInsertPt(U), Expr));
+ }
+ }
+ return Modified;
+}
+
+bool ExpandConstantExpr::runOnFunction(Function &Func) {
+ bool Modified = false;
+ for (llvm::Function::iterator BB = Func.begin(), E = Func.end();
+ BB != E;
+ ++BB) {
+ for (BasicBlock::InstListType::iterator Inst = BB->begin(), E = BB->end();
+ Inst != E;
+ ++Inst) {
+ Modified |= expandInstruction(Inst);
+ }
+ }
+ return Modified;
+}
+
+FunctionPass *llvm::createExpandConstantExprPass() {
+ return new ExpandConstantExpr();
+}
« no previous file with comments | « lib/Transforms/NaCl/ExpandByVal.cpp ('k') | lib/Transforms/NaCl/ExpandCtors.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698