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

Unified Diff: src/IceDefs.h

Issue 1341423002: Reflow comments to use the full width. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Fix spelling and rebase Created 5 years, 3 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 | « src/IceConverter.cpp ('k') | src/IceELFObjectWriter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/IceDefs.h
diff --git a/src/IceDefs.h b/src/IceDefs.h
index d1ad81cfd3794f3a5e2f2a433d9ed948fe98705c..a38da03e290511a53f673b3e47e08c90c6d4194a 100644
--- a/src/IceDefs.h
+++ b/src/IceDefs.h
@@ -9,7 +9,7 @@
///
/// \file
/// This file declares various useful types and classes that have widespread use
-/// across Subzero. Every Subzero source file is expected to include IceDefs.h.
+/// across Subzero. Every Subzero source file is expected to include IceDefs.h.
///
//===----------------------------------------------------------------------===//
@@ -99,8 +99,8 @@ inline bool operator!=(const CfgLocalAllocator<T> &,
}
// makeUnique should be used when memory is expected to be allocated from the
-// heap (as opposed to allocated from some Allocator.) It is intended to be used
-// instead of new.
+// heap (as opposed to allocated from some Allocator.) It is intended to be
+// used instead of new.
//
// The expected usage is as follows
//
@@ -140,8 +140,8 @@ static std::unique_ptr<T> makeUnique(Args &&... TheArgs) {
using IceString = std::string;
using InstList = llvm::ilist<Inst>;
-// Ideally PhiList would be llvm::ilist<InstPhi>, and similar for
-// AssignList, but this runs into issues with SFINAE.
+// Ideally PhiList would be llvm::ilist<InstPhi>, and similar for AssignList,
+// but this runs into issues with SFINAE.
using PhiList = InstList;
using AssignList = InstList;
@@ -155,18 +155,18 @@ using ConstantList = std::vector<Constant *>;
using FunctionDeclarationList = std::vector<FunctionDeclaration *>;
using VariableDeclarationList = std::vector<VariableDeclaration *>;
-/// SizeT is for holding small-ish limits like number of source
-/// operands in an instruction. It is used instead of size_t (which
-/// may be 64-bits wide) when we want to save space.
+/// SizeT is for holding small-ish limits like number of source operands in an
+/// instruction. It is used instead of size_t (which may be 64-bits wide) when
+/// we want to save space.
using SizeT = uint32_t;
-/// InstNumberT is for holding an instruction number. Instruction
-/// numbers are used for representing Variable live ranges.
+/// InstNumberT is for holding an instruction number. Instruction numbers are
+/// used for representing Variable live ranges.
using InstNumberT = int32_t;
-/// A LiveBeginEndMapEntry maps a Variable::Number value to an
-/// Inst::Number value, giving the instruction number that begins or
-/// ends a variable's live range.
+/// A LiveBeginEndMapEntry maps a Variable::Number value to an Inst::Number
+/// value, giving the instruction number that begins or ends a variable's live
+/// range.
using LiveBeginEndMapEntry = std::pair<SizeT, InstNumberT>;
using LiveBeginEndMap =
std::vector<LiveBeginEndMapEntry, CfgLocalAllocator<LiveBeginEndMapEntry>>;
@@ -175,9 +175,8 @@ using LivenessBV = llvm::BitVector;
using TimerStackIdT = uint32_t;
using TimerIdT = uint32_t;
-/// Use alignas(MaxCacheLineSize) to isolate variables/fields that
-/// might be contended while multithreading. Assumes the maximum cache
-/// line size is 64.
+/// Use alignas(MaxCacheLineSize) to isolate variables/fields that might be
+/// contended while multithreading. Assumes the maximum cache line size is 64.
enum { MaxCacheLineSize = 64 };
// Use ICE_CACHELINE_BOUNDARY to force the next field in a declaration
// list to be aligned to the next cache line.
@@ -191,15 +190,15 @@ using RelocOffsetT = int32_t;
enum { RelocAddrSize = 4 };
enum LivenessMode {
- /// Basic version of live-range-end calculation. Marks the last uses
- /// of variables based on dataflow analysis. Records the set of
- /// live-in and live-out variables for each block. Identifies and
- /// deletes dead instructions (primarily stores).
+ /// Basic version of live-range-end calculation. Marks the last uses of
+ /// variables based on dataflow analysis. Records the set of live-in and
+ /// live-out variables for each block. Identifies and deletes dead
+ /// instructions (primarily stores).
Liveness_Basic,
- /// In addition to Liveness_Basic, also calculate the complete
- /// live range for each variable in a form suitable for interference
- /// calculation and register allocation.
+ /// In addition to Liveness_Basic, also calculate the complete live range for
+ /// each variable in a form suitable for interference calculation and register
+ /// allocation.
Liveness_Intervals
};
@@ -244,10 +243,10 @@ using GlobalLockType = std::mutex;
enum ErrorCodes { EC_None = 0, EC_Args, EC_Bitcode, EC_Translation };
-/// Wrapper around std::error_code for allowing multiple errors to be
-/// folded into one. The current implementation keeps track of the
-/// first error, which is likely to be the most useful one, and this
-/// could be extended to e.g. collect a vector of errors.
+/// Wrapper around std::error_code for allowing multiple errors to be folded
+/// into one. The current implementation keeps track of the first error, which
+/// is likely to be the most useful one, and this could be extended to e.g.
+/// collect a vector of errors.
class ErrorCode : public std::error_code {
ErrorCode(const ErrorCode &) = delete;
ErrorCode &operator=(const ErrorCode &) = delete;
« no previous file with comments | « src/IceConverter.cpp ('k') | src/IceELFObjectWriter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698