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

Unified Diff: lib/Bitcode/NaCl/Reader/NaClBitCodes.cpp

Issue 154603002: Make pnacl-bccompress add abbreviations for obvious constants. (Closed) Base URL: http://git.chromium.org/native_client/pnacl-llvm.git@master
Patch Set: Fixs nits associated with Patch Set 4. Created 6 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/Bitcode/NaCl/Reader/CMakeLists.txt ('k') | lib/Bitcode/NaCl/Reader/NaClBitcodeDist.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/Bitcode/NaCl/Reader/NaClBitCodes.cpp
diff --git a/lib/Bitcode/NaCl/Reader/NaClBitCodes.cpp b/lib/Bitcode/NaCl/Reader/NaClBitCodes.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..1299e938634a7400d3557914351379bd9f55fd38
--- /dev/null
+++ b/lib/Bitcode/NaCl/Reader/NaClBitCodes.cpp
@@ -0,0 +1,97 @@
+//===- NaClBitcodeHeader.cpp ----------------------------------------------===//
+// PNaCl bitcode header reader.
+//
+// The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+//
+// Implementation of Bitcode abbrevations.
+//
+//===----------------------------------------------------------------------===//
+
+#include "llvm/Bitcode/NaCl/NaClBitCodes.h"
+#include "llvm/Support/raw_ostream.h"
+#include "llvm/Support/ErrorHandling.h"
+
+using namespace llvm;
+
+void NaClBitCodeAbbrevOp::Print(raw_ostream& Stream) const {
+ if (isLiteral()) {
+ Stream << getLiteralValue();
+ } else if (isEncoding()) {
+ switch (getEncoding()) {
+ case Fixed:
+ Stream << "Fixed(" << getEncodingData() << ")";
+ break;
+ case VBR:
+ Stream << "VBR(" << getEncodingData() << ")";
+ break;
+ case Array:
+ Stream << "Array";
+ break;
+ case Char6:
+ Stream << "Char6";
+ break;
+ case Blob:
+ Stream << "Blob";
+ break;
+ default:
+ llvm_unreachable("Unknown bitcode abbreviation operator");
+ Stream << "??"; // In case asserts are turned off.
+ break;
+ }
+ } else {
+ llvm_unreachable("Unknown bitcode abbreviation operator");
+ Stream << "??"; // In case asserts are turned off.
+ }
+}
+
+static void PrintExpression(raw_ostream &Stream,
+ const NaClBitCodeAbbrev *Abbrev,
+ unsigned &Index) {
+ // Bail out early, in case we are incrementally building the
+ // expression and the argument is not available yet.
+ if (Index >= Abbrev->getNumOperandInfos()) return;
+
+ const NaClBitCodeAbbrevOp &Op = Abbrev->getOperandInfo(Index);
+ Op.Print(Stream);
+ if (unsigned NumArgs = Op.NumArguments()) {
+ Stream << "(";
+ for (unsigned i = 0; i < NumArgs; ++i) {
+ ++Index;
+ if (i > 0) Stream << ",";
+ PrintExpression(Stream, Abbrev, Index);
+ }
+ Stream << ")";
+ }
+}
+
+void NaClBitCodeAbbrev::Print(raw_ostream &Stream) const {
+ Stream << "[";
+ for (unsigned i = 0; i < getNumOperandInfos(); ++i) {
+ if (i > 0) Stream << ", ";
+ PrintExpression(Stream, this, i);
+ }
+ Stream << "]\n";
+}
+
+NaClBitCodeAbbrev *NaClBitCodeAbbrev::Simplify() const {
+ NaClBitCodeAbbrev *Abbrev = new NaClBitCodeAbbrev();
+ for (unsigned i = 0; i < OperandList.size(); ++i) {
+ const NaClBitCodeAbbrevOp &Op = OperandList[i];
+ // Simplify if possible. Currently, the only simplification known
+ // is to remove unnecessary operands appearing immediately before an
+ // array operator. That is, apply the simplification:
+ // Op Array(Op) -> Array(Op)
+ assert(!Op.isArrayOp() || i == OperandList.size()-2);
+ while (Op.isArrayOp() && !Abbrev->OperandList.empty() &&
+ Abbrev->OperandList.back() == OperandList[i+1]) {
+ Abbrev->OperandList.pop_back();
+ }
+ Abbrev->OperandList.push_back(Op);
+ }
+ return Abbrev;
+}
« no previous file with comments | « lib/Bitcode/NaCl/Reader/CMakeLists.txt ('k') | lib/Bitcode/NaCl/Reader/NaClBitcodeDist.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698