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

Unified Diff: src/IceTargetLoweringARM32.h

Issue 1216963007: Doxygenize the documentation comments (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Created 5 years, 6 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
Index: src/IceTargetLoweringARM32.h
diff --git a/src/IceTargetLoweringARM32.h b/src/IceTargetLoweringARM32.h
index fdb025f883bea6185eb82844e9565b0356e5f5d7..c3382c481580f99636101520b28e25119dc43773 100644
--- a/src/IceTargetLoweringARM32.h
+++ b/src/IceTargetLoweringARM32.h
@@ -66,9 +66,9 @@ public:
void addProlog(CfgNode *Node) override;
void addEpilog(CfgNode *Node) override;
- // Ensure that a 64-bit Variable has been split into 2 32-bit
- // Variables, creating them if necessary. This is needed for all
- // I64 operations.
+ /// Ensure that a 64-bit Variable has been split into 2 32-bit
+ /// Variables, creating them if necessary. This is needed for all
+ /// I64 operations.
void split64(Variable *Var);
Operand *loOperand(Operand *Operand);
Operand *hiOperand(Operand *Operand);
@@ -117,10 +117,10 @@ protected:
enum OperandLegalization {
Legal_None = 0,
- Legal_Reg = 1 << 0, // physical register, not stack location
- Legal_Flex = 1 << 1, // A flexible operand2, which can hold rotated
- // small immediates, or shifted registers.
- Legal_Mem = 1 << 2, // includes [r0, r1 lsl #2] as well as [sp, #12]
+ Legal_Reg = 1 << 0, /// physical register, not stack location
+ Legal_Flex = 1 << 1, /// A flexible operand2, which can hold rotated
+ /// small immediates, or shifted registers.
+ Legal_Mem = 1 << 2, /// includes [r0, r1 lsl #2] as well as [sp, #12]
Legal_All = ~Legal_None
};
typedef uint32_t LegalMask;
@@ -134,7 +134,7 @@ protected:
Variable *copyToReg(Operand *Src, int32_t RegNum = Variable::NoRegister);
void alignRegisterPow2(Variable *Reg, uint32_t Align);
- // Returns a vector in a register with the given constant entries.
+ /// Returns a vector in a register with the given constant entries.
Variable *makeVectorOfZeros(Type Ty, int32_t RegNum = Variable::NoRegister);
void makeRandomRegisterPermutation(
@@ -207,9 +207,9 @@ protected:
CondARM32::Cond Pred = CondARM32::AL) {
Context.insert(InstARM32Mla::create(Func, Dest, Src0, Src1, Acc, Pred));
}
- // If Dest=nullptr is passed in, then a new variable is created,
- // marked as infinite register allocation weight, and returned
- // through the in/out Dest argument.
+ /// If Dest=nullptr is passed in, then a new variable is created,
+ /// marked as infinite register allocation weight, and returned
+ /// through the in/out Dest argument.
void _mov(Variable *&Dest, Operand *Src0,
CondARM32::Cond Pred = CondARM32::AL,
int32_t RegNum = Variable::NoRegister) {
@@ -223,8 +223,8 @@ protected:
NewInst->setDestNonKillable();
Context.insert(NewInst);
}
- // The Operand can only be a 16-bit immediate or a ConstantRelocatable
- // (with an upper16 relocation).
+ /// The Operand can only be a 16-bit immediate or a ConstantRelocatable
+ /// (with an upper16 relocation).
void _movt(Variable *Dest, Operand *Src0,
CondARM32::Cond Pred = CondARM32::AL) {
Context.insert(InstARM32Movt::create(Func, Dest, Src0, Pred));
@@ -313,16 +313,16 @@ protected:
VarList PhysicalRegisters[IceType_NUM];
static IceString RegNames[];
- // Helper class that understands the Calling Convention and register
- // assignments. The first few integer type parameters can use r0-r3,
- // regardless of their position relative to the floating-point/vector
- // arguments in the argument list. Floating-point and vector arguments
- // can use q0-q3 (aka d0-d7, s0-s15). Technically, arguments that can
- // start with registers but extend beyond the available registers can be
- // split between the registers and the stack. However, this is typically
- // for passing GPR structs by value, and PNaCl transforms expand this out.
- //
- // Also, at the point before the call, the stack must be aligned.
+ /// Helper class that understands the Calling Convention and register
+ /// assignments. The first few integer type parameters can use r0-r3,
+ /// regardless of their position relative to the floating-point/vector
+ /// arguments in the argument list. Floating-point and vector arguments
+ /// can use q0-q3 (aka d0-d7, s0-s15). Technically, arguments that can
+ /// start with registers but extend beyond the available registers can be
+ /// split between the registers and the stack. However, this is typically
+ /// for passing GPR structs by value, and PNaCl transforms expand this out.
+ ///
+ /// Also, at the point before the call, the stack must be aligned.
class CallingConv {
CallingConv(const CallingConv &) = delete;
CallingConv &operator=(const CallingConv &) = delete;

Powered by Google App Engine
This is Rietveld 408576698