Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .codespellrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
[codespell]
# Ref: https://github.com/codespell-project/codespell#using-a-config-file
skip = .git,.gitignore,.gitattributes,.codespellrc
check-hidden = true
# ignore-regex =
# ignore-words-list =
23 changes: 23 additions & 0 deletions .github/workflows/codespell.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# Codespell configuration is within .codespellrc
---
name: Codespell

on:
push:
branches: [master]
pull_request:
branches: [master]

permissions:
contents: read

jobs:
codespell:
name: Check for spelling errors
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v7
- name: Codespell
uses: codespell-project/actions-codespell@v2.2
2 changes: 1 addition & 1 deletion ci/ci_common/run-spec-impl.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ local std_get = (import "../../ci/ci_common/common-utils.libsonnet").std_get;
local onlyMatrixKeys = union == known_fields;
local noMatrixKeys = inter == [];
if isLeaf then
assert noMatrixKeys: "unexected platform spec keys in build definition %s (context: %s, stack: %s)" % [inter, context, stack];
assert noMatrixKeys: "unexpected platform spec keys in build definition %s (context: %s, stack: %s)" % [inter, context, stack];
obj
else
assert onlyMatrixKeys : "unexpected keys in platform spec %s, expected %s (context: %s, stack: %s)" % [diff, known_fields, context, stack];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ boolean canBeCalledWithoutErrorMessage() {
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("Method info for ").append(method).append(System.lineSeparator());
sb.append("\tValid asserion usage ?").append(correctAssertionMethod).append(System.lineSeparator());
sb.append("\tValid assertion usage ?").append(correctAssertionMethod).append(System.lineSeparator());
sb.append("\troot?").append(root).append(System.lineSeparator());
sb.append("\tcalls assertion(transitively)?").append(callsAssertionTransitively).append(System.lineSeparator());
sb.append("\tValid use as assertion method based on paths?").append(allPathsAssertionDominated).append(System.lineSeparator());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ protected DataSource createDataSource(URL bigv) throws IOException {
FileChannel fch2 = FileChannel.open(f.toPath(), StandardOpenOption.READ);
return new BinarySource(null, fch2);
} catch (URISyntaxException ex) {
throw new AssertionError("Canot convert " + bigv, ex);
throw new AssertionError("Cannot convert " + bigv, ex);
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -755,7 +755,7 @@ protected void lowerIntegerDivRem(IntegerDivRemNode n, LoweringTool tool) {
} else if (n instanceof SignedRemNode) {
divRem = graph.addOrUnique(SignedFloatingIntegerRemNode.create(dividend, divisor, NodeView.DEFAULT, guard, divisionOverflowIsJVMSCompliant));
} else {
throw GraalError.shouldNotReachHere("Unkown division node " + n); // ExcludeFromJacocoGeneratedReport
throw GraalError.shouldNotReachHere("Unknown division node " + n); // ExcludeFromJacocoGeneratedReport
}
n.replaceAtUsages(divRem);
graph.replaceFixedWithFloating(n, divRem);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -602,7 +602,7 @@ public AllocatableValue spillSlot() {

public void setSpillSlot(AllocatableValue slot) {
assert LIRValueUtil.isStackSlotValue(slot);
assert splitParent().spillSlot == null || LIRValueUtil.isVirtualStackSlot(splitParent().spillSlot) && isStackSlot(slot) : "connot overwrite existing spill slot";
assert splitParent().spillSlot == null || LIRValueUtil.isVirtualStackSlot(splitParent().spillSlot) && isStackSlot(slot) : "cannot overwrite existing spill slot";
splitParent().spillSlot = slot;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ private void optimizeInterval(LIRInsertionBuffer[] insertionBuffers, Interval in
if (defBlock.getLoopDepth() < spillBlock.getLoopDepth()) {
spillBlock = moveSpillOutOfLoop(defBlock, spillBlock);
}
debug.log(DebugContext.VERBOSE_LEVEL, "Spill block candidate (after loop optimizaton): %s", spillBlock);
debug.log(DebugContext.VERBOSE_LEVEL, "Spill block candidate (after loop optimization): %s", spillBlock);

/*
* The spill block is the begin of the first split child (aka the value is on the
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ protected void resolveCollectMappings(BasicBlock<?> fromBlock, BasicBlock<?> toB

// visit all variables for which the liveAtEdge bit is set
for (int operandNum = liveAtEdge.iterateValues(0); operandNum >= 0; operandNum = liveAtEdge.iterateValues(operandNum + 1)) {
assert operandNum < numOperands : "live information set for not exisiting interval";
assert operandNum < numOperands : "live information set for not existing interval";
assert allocator.getBlockData(fromBlock).liveOut.get(operandNum) && allocator.getBlockData(toBlock).liveIn.get(operandNum) : "interval not live at this edge";

Interval fromInterval = allocator.splitChildAtOpId(allocator.intervalFor(operandNum), fromBlockLastInstructionId, LIRInstruction.OperandMode.DEF);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public void emitCode(CompilationResultBuilder crb, AMD64MacroAssembler masm) {
masm.prefetchw(address.toAddress(masm));
break;
default:
throw GraalError.shouldNotReachHere("unspported prefetch op " + instr); // ExcludeFromJacocoGeneratedReport
throw GraalError.shouldNotReachHere("unsupported prefetch op " + instr); // ExcludeFromJacocoGeneratedReport

}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1052,7 +1052,7 @@ private HIRBlock earliestBlockForGuard(GuardNode guard, CFGLoop<HIRBlock> forced
b = newAnchorEarliest;
}

debug.log("earliestBlockForGuard(%s) inital anchor : %s, condition : %s condition's earliest %s", guard, anchor, guard.getCondition(), conditionEarliest.getBeginNode());
debug.log("earliestBlockForGuard(%s) initial anchor : %s, condition : %s condition's earliest %s", guard, anchor, guard.getCondition(), conditionEarliest.getBeginNode());

double minFrequency = anchorEarliest.getRelativeFrequency();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -819,7 +819,7 @@ private EconomicMap<String, Object> callsiteAsJSONMap(InliningLog.Callsite calls
}
if (callsite.getOverriddenParent() != null) {
EconomicMap<String, Object> parentMap = replacements.get(callsite.getOverriddenParent());
assert parentMap != null : "there must already exist a JSON map for the overriden parent";
assert parentMap != null : "there must already exist a JSON map for the overridden parent";
List<Object> parentInvokesProperty = (List<Object>) parentMap.get(INVOKES_PROPERTY);
if (parentInvokesProperty == null) {
parentInvokesProperty = new ArrayList<>();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ public static boolean verify(ControlFlowGraph cfg) {
CFGLoop<?> blockLoop = block.getLoop();
while (blockLoop != null) {
blockLoop = blockLoop.getParent();
assert blockLoop != loop : "Parent loop must be different than loop that is exitted " + Assertions.errorMessageContext("blockLoop", blockLoop, "loop", loop);
assert blockLoop != loop : "Parent loop must be different than loop that is exited " + Assertions.errorMessageContext("blockLoop", blockLoop, "loop", loop);
}
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,9 +87,9 @@ public static class Options {
public static final OptionKey<Double> LoopUnswitchFrequencyBoost = new OptionKey<>(40.0);
@Option(help = "Minimum value for the frequency factor of an invariant.", type = OptionType.Debug)
public static final OptionKey<Double> LoopUnswitchFrequencyMinFactor = new OptionKey<>(0.05);
@Option(help = "Maximun value for the frequency factor of an invariant.", type = OptionType.Debug)
@Option(help = "Maximum value for the frequency factor of an invariant.", type = OptionType.Debug)
public static final OptionKey<Double> LoopUnswitchFrequencyMaxFactor = new OptionKey<>(0.95);
@Option(help = "Lower bound for the minimun frequency of an invariant condition to be unswitched.", type = OptionType.Debug)
@Option(help = "Lower bound for the minimum frequency of an invariant condition to be unswitched.", type = OptionType.Debug)
public static final OptionKey<Double> LoopUnswitchMinSplitFrequency = new OptionKey<>(1.0);
@Option(help = "Default frequency for loops with unknown local frequency.", type = OptionType.Debug)
public static final OptionKey<Double> DefaultLoopFrequency = new OptionKey<>(100.0);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ public static UnswitchingDecision yes(List<ControlSplitNode> controlSplits) {
private final boolean isTrivial;

private UnswitchingDecision(List<ControlSplitNode> controlSplits, boolean isTrivial) {
assert !isTrivial || controlSplits != null : "An unswitching desision cannot be trivial but have not control split node";
assert !isTrivial || controlSplits != null : "An unswitching decision cannot be trivial but have not control split node";

this.controlSplits = controlSplits;
this.isTrivial = isTrivial;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -555,7 +555,7 @@ private static BlockMap<List<Node>> transplantScheduleResult(StructuredGraph gra
}
}
} else {
throw GraalError.shouldNotReachHere("Unkown block source");
throw GraalError.shouldNotReachHere("Unknown block source");
}
newBlockToNodesMap.put(b, blockNodes);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1000,9 +1000,9 @@ static boolean checkInlineeFrameState(Invoke invoke, ResolvedJavaMethod inlinedM
// Normal inlining expects all outermost inlinee frame states to
// denote the inlinee method
} else if (method.equals(invoke.callTarget().targetMethod())) {
GraalError.shouldNotReachHere("method subsitutions are gone"); // ExcludeFromJacocoGeneratedReport
GraalError.shouldNotReachHere("method substitutions are gone"); // ExcludeFromJacocoGeneratedReport
} else if (method.getName().equals(inlinedMethod.getName())) {
GraalError.shouldNotReachHere("method subsitutions are gone"); // ExcludeFromJacocoGeneratedReport
GraalError.shouldNotReachHere("method substitutions are gone"); // ExcludeFromJacocoGeneratedReport
} else {
throw new AssertionError(String.format("inlinedMethod=%s frameState.method=%s frameState=%s invoke.method=%s", inlinedMethod, method, frameState,
invoke.callTarget().targetMethod()));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public Decision isWorthInlining(Replacements replacements, MethodInvocation invo
if (callTarget instanceof MethodCallTargetNode) {
ResolvedJavaMethod calleeMethod = callTarget.targetMethod();
if (replacements.hasSubstitution(calleeMethod, options)) {
return Decision.YES.withReason(isTracing, "has a method subtitution");
return Decision.YES.withReason(isTracing, "has a method substitution");
}
}
return Decision.NO.withReason(isTracing, "does not have a method substitution");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ private void printNode(Node node, boolean unscheduled) {
}
out.println("=== Inputs ===");
printNamedNodes(node, node.inputPositions().iterator(), "", "\n", null);
out.println("=== Succesors ===");
out.println("=== Successors ===");
printNamedNodes(node, node.successorPositions().iterator(), "", "\n", null);
out.println("=== Usages ===");
if (!node.hasNoUsages()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ public boolean mayExpandToLoop() {
// will become a loop
return true;
default:
throw GraalError.shouldNotReachHere("Unkown snippet type " + snippet); // ExcludeFromJacocoGeneratedReport
throw GraalError.shouldNotReachHere("Unknown snippet type " + snippet); // ExcludeFromJacocoGeneratedReport
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -605,7 +605,7 @@ private boolean tryVectorPackedComparisonZero(Condition condition, Variable resu
}

private Variable emitVectorPackedComparison(Condition condition, Value vectorA, Value vectorB, boolean unorderedIsTrue) {
assert isVectorKind(vectorA) : "A must be a vactor graph " + vectorA;
assert isVectorKind(vectorA) : "A must be a vector graph " + vectorA;

Variable result = getVectorComparisonResult(vectorA);
/* First try to emit a comparison against zero. */
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public void setupProvider() {
if (provider != null && !provider.isEmpty()) {
prov = Security.getProvider(provider);
if (prov == null) {
throw new RuntimeException("Can't find prodiver \"" + provider + "\"");
throw new RuntimeException("Can't find provider \"" + provider + "\"");
}
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@
"title": "Virtual invoke profiles",
"type": "array",
"default": [],
"description": "Information about the run-time receiver types of virutal invokes.",
"description": "Information about the run-time receiver types of virtual invokes.",
"items":{
"$id": "#root/virtualInvokeProfiles/items",
"title": "Items",
Expand All @@ -314,7 +314,7 @@
"$id": "#root/virtualInvokeProfiles/items/records",
"title": "Records",
"type": "array",
"description": "Array of integer values containing a series of pairs: type id and number of times the virutal call receiver was of that type.",
"description": "Array of integer values containing a series of pairs: type id and number of times the virtual call receiver was of that type.",
"items":{
"$id": "#root/virtualInvokeProfiles/items/records/items",
"title": "Items",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@
"title": "Virtual invoke profiles",
"type": "array",
"default": [],
"description": "Information about the run-time receiver types of virutal invokes.",
"description": "Information about the run-time receiver types of virtual invokes.",
"items":{
"$id": "#root/virtualInvokeProfiles/items",
"title": "Items",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ public Operand<R, C, M, F>[] extract() {

void procSize(int modif) {
size += modif;
verifyGuarantee(size <= stack.length, "insufficent stack size: " + stack.length);
verifyGuarantee(size <= stack.length, "insufficient stack size: " + stack.length);
verifyGuarantee(size >= 0, "invalid stack access: " + size);
}

Expand Down
4 changes: 2 additions & 2 deletions espresso/src/com.oracle.truffle.espresso.eden/src/eden.c
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ void *dlopen(const char *filename, int flags) {

// glibc yields a loading error if the file doesn't exists.
if (is_absolute_path(filename) && access(filename, R_OK) != 0) {
LOG("dlopen(%s, %d): File not accesible\n", filename, flags);
LOG("dlopen(%s, %d): File not accessible\n", filename, flags);
return NULL;
}

Expand Down Expand Up @@ -206,7 +206,7 @@ static __attribute__((constructor)) void initialize(void) {
FATAL("initialize Error obtaining namespace (dlinfo): %s\n", dlerror());
}
if (!namespace_id) {
FATAL("initialize libeden.so should't be loaded in the default namespace\n");
FATAL("initialize libeden.so shouldn't be loaded in the default namespace\n");
}
LOG("initialize &__ctype_b_loc: %p\n", &__ctype_b_loc);
LOG("initialize *__ctype_b_loc() = %p\n", *__ctype_b_loc());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ public static Packet fromByteArray(byte[] b) throws IOException, ConnectionClose
int b3 = b[3] & 0xff;
int len = ((b0 << 24) | (b1 << 16) | (b2 << 8) | (b3 << 0));
if (len != b.length) {
throw new IOException("length size mis-match");
throw new IOException("length size mismatch");
}

int b4 = b[4] & 0xff;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ public void writePacket(byte[] b) throws IOException {
* Check that the byte array contains the complete packet
*/
if (len > b.length) {
throw new IllegalArgumentException("length mis-match");
throw new IllegalArgumentException("length mismatch");
}
/*
* Send the packet (ignoring any bytes that follow the packet in the byte array).
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ private static Map<String, String> decodeEnv(byte[] envBlockArr, int envc, Espre
String value = envStr.substring(equalsIndex + 1);
envMap.put(key, value);
} else {
throw Throw.throwIOException("The byte array encoding for the Enviornment Map Entry didnt include =", ctx);
throw Throw.throwIOException("The byte array encoding for the Environment Map Entry didn't include =", ctx);
}
i++; // skip the null-terminator
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ public static void register(int selectorId, int fd, int newEvents,
* sun.nio.ch.EPollSelectorImpl.processUpdateQueue error numbers from the native
* sun.nio.ch.EPoll.ctl are just ignored. So we do the same here.
*/
LibsState.getLogger().log(Level.WARNING, "In io.register the following exception was ingored: ", e);
LibsState.getLogger().log(Level.WARNING, "In io.register the following exception was ignored: ", e);
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ private static void reconstruct(StringsProvider reader, ByteBuffer input, ByteBu
ByteBuffer pkg = reader.getRawString(CompressedIndexes.readInt(input));
ByteBuffer clazz = reader.getRawString(CompressedIndexes.readInt(input));
if (input.position() > indiciesLimit) {
throw new RuntimeException("Missing indicies");
throw new RuntimeException("Missing indices");
}
// 'L' (pkg '/')? clazz
output.put((byte) 'L');
Expand All @@ -195,7 +195,7 @@ private static void reconstruct(StringsProvider reader, ByteBuffer input, ByteBu
transfert(desc, runStart, output, runLength);
}
if (input.position() < indiciesLimit) {
BasicImageReader.LOGGER.warning("StringSharingDecompressor: " + (indiciesLimit - input.position()) + " indicies bytes remain unused after reconstructing descriptor");
BasicImageReader.LOGGER.warning("StringSharingDecompressor: " + (indiciesLimit - input.position()) + " indices bytes remain unused after reconstructing descriptor");
input.position(indiciesLimit);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3796,7 +3796,7 @@ public int JNI_GetCreatedJavaVMs(@Pointer TruffleObject vmBufPtr, int bufLen, @P
StaticObject guestName = meta.java_lang_Module_name.getObject(module);
if (StaticObject.isNull(guestName)) {
profiler.profile(4);
throw meta.throwExceptionWithMessage(meta.java_lang_IllegalArgumentException, "modue name cannot be null");
throw meta.throwExceptionWithMessage(meta.java_lang_IllegalArgumentException, "module name cannot be null");
}

String hostName = meta.toHostString(guestName);
Expand Down
Loading
Loading