You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by va...@apache.org on 2008/06/10 11:12:17 UTC
svn commit: r666024 - in /harmony/enhanced/drlvm/trunk/vm:
jitrino/src/codegenerator/ia32/ port/src/encoder/ia32_em64t/
Author: varlax
Date: Tue Jun 10 02:12:17 2008
New Revision: 666024
URL: http://svn.apache.org/viewvc?rev=666024&view=rev
Log:
Fixed spotted typos in comments
Modified:
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32ConstraintsResolver.cpp
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32EarlyPropagation.cpp
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Inst.h
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32InstCodeSelector.cpp
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Printer.cpp
harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32RegAlloc3.cpp
harmony/enhanced/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_defs.h
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32ConstraintsResolver.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32ConstraintsResolver.cpp?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32ConstraintsResolver.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32ConstraintsResolver.cpp Tue Jun 10 02:12:17 2008
@@ -157,10 +157,10 @@
* the calculated constraint is set to the constraint
* preserving operand values during exception throwing
* This constraint is returned by getDispatchEntryConstraint
- * In fact this is the constriant for the DRL calling convention
+ * In fact this is the constraint for the DRL calling convention
*
* This is done in calculateStartupOpndConstraints()
- * Originally all calculateed constraints are equial to Initial constraints
+ * Originally all calculated constraints are equal to Initial constraints
*
* 4) Walks through all basic blocks collected and arranged at step 1
* in resolveConstraints()
@@ -182,14 +182,14 @@
*
* To do this already collected calculated constraint of
* either original operand or its current replacement is anded
- * with instruction constraint for this operand occurence and
+ * with instruction constraint for this operand occurrence and
* if the result is null, new operand is created and substituted instead
*
- * 4.1.1.1) All def operands of the isntruction are traversed
+ * 4.1.1.1) All def operands of the instruction are traversed
* and operand splitting is performed after the instruction (when necessary)
* def&use cases are also handled during this step
* 4.1.1.2) If the instruction is CALL, all hovering operands of
- * the isntruction are traversed.
+ * the instruction are traversed.
*
* Hovering operands are operands which are live across a call site and are not
* redefined in the call site
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32EarlyPropagation.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32EarlyPropagation.cpp?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32EarlyPropagation.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32EarlyPropagation.cpp Tue Jun 10 02:12:17 2008
@@ -35,7 +35,7 @@
//========================================================================================
/**
* class EarlyPropagation implements a simple algorithm of constant and copy propagation.
- * It works only with operands which have single defintions.
+ * It works only with operands which have single definitions.
* In addition to constants it also propagates heap reads.
*
* The main goal of the pass is to reduce register pressure.
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Inst.h
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Inst.h?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Inst.h (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Inst.h Tue Jun 10 02:12:17 2008
@@ -84,7 +84,7 @@
DefScope_Null=0,
/** The operand has single definition */
DefScope_Temporary,
- /** The operand has multiple defs all within one basic block (occures after convertion into 2-operand form) */
+ /** The operand has multiple defs all within one basic block (occurs after conversion into 2-operand form) */
DefScope_SemiTemporary,
/** The operand has merging defs */
DefScope_Variable,
@@ -94,7 +94,7 @@
enum ConstraintKind{
/** An additional constraint assigned during Opnd creation */
ConstraintKind_Initial=0,
- /** A constraint calculated in the contstraint resolver from instruction properties */
+ /** A constraint calculated in the constraint resolver from instruction properties */
ConstraintKind_Calculated=1,
/** A constraint defining assigned physical location of an operand */
ConstraintKind_Location=2,
@@ -127,7 +127,7 @@
Kind_TypeRuntimeId,
/** The value of the operand is [0]->NamedType::getRuntimeIdentifier() */
Kind_MethodRuntimeId,
- /** The value of the operand is [1], but the information can be used to serialize/deserizalize
+ /** The value of the operand is [1], but the information can be used to serialize/deserialize
this value: [0] - Type * - the containing class, [1] - string token */
Kind_StringDescription,
/** The value of the operand is [0]->ObjectType::getObjectSize() */
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32InstCodeSelector.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32InstCodeSelector.cpp?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32InstCodeSelector.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32InstCodeSelector.cpp Tue Jun 10 02:12:17 2008
@@ -2396,7 +2396,7 @@
CallInst * callInst = irManager.newRuntimeHelperCallInst(VM_RT_LDC_STRING, 2, helperOpnds, retOpnd);
appendInsts(callInst);
} else {
- // this optimized version is based on determinig item address at compile time.
+ // this optimized version is based on determining item address at compile time.
// Respective compile time helper (class_get_const_string_intern_addr) is available for strings.
// Similar function for literal constants is not ready.
// TODO: rewrite this as soon as the helper for loading ref addr at compile time is ready.
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Printer.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Printer.cpp?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Printer.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32Printer.cpp Tue Jun 10 02:12:17 2008
@@ -517,7 +517,7 @@
case Opnd::RuntimeInfo::Kind_MethodRuntimeId:
{
MethodDesc * md=(MethodDesc*)info->getValue(0);
- os<<"vtso:";
+ os<<"mh:";
os<<md->getParentType()->getName()<<"."<<md->getName();
}break;
case Opnd::RuntimeInfo::Kind_EM_ProfileAccessInterface:
Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32RegAlloc3.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32RegAlloc3.cpp?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32RegAlloc3.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32RegAlloc3.cpp Tue Jun 10 02:12:17 2008
@@ -689,7 +689,7 @@
}
// Iterate over all instructions in CFG and calculate which operands
-// are live simultaneouesly (result stored in matrix)
+// live simultaneously (result stored in matrix)
BitSet lives(mm, opandcount);
@@ -744,7 +744,7 @@
if (graph[x].oproles->empty())
graph[x].ignore = true;
-// Connect nodes that represent simultaneouesly live operands
+// Connect nodes that represent simultaneously live operands
for (unsigned x1 = 1; x1 < graphsize; ++x1)
for (unsigned x2 = 0; x2 < x1; ++x2)
@@ -980,8 +980,8 @@
}
-// Colalesce graph nodes (x0) and (x1) and the corresponding operands.
-// Node (x1) not to be used anymore, (x0) must be used unstead.
+// Coalesce graph nodes (x0) and (x1) and the corresponding operands.
+// Node (x1) not to be used anymore, (x0) must be used instead.
// Note that (x1) remains in the graph (must be ignored)
//
void RegAlloc3::coalesce (int* opandmap, BoolMatrix& matrix, int x0, int x1)
Modified: harmony/enhanced/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_defs.h
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_defs.h?rev=666024&r1=666023&r2=666024&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_defs.h (original)
+++ harmony/enhanced/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_defs.h Tue Jun 10 02:12:17 2008
@@ -161,7 +161,7 @@
// Gregory -
// It is critical that all register indexes (3rd number) inside of the
// following table go in ascending order. That is R8 goes after
-// RDI. It is necessary for decoder when extending registers from RAX-DRI
+// RDI. It is necessary for decoder when extending registers from RAX-RDI
// to R8-R15 by simply adding 8 to the index on EM64T architecture
enum RegName {