Merge branch 'hashx_sizet_mr' into 'main'

hashx: Fix a few more compiler warnings

See merge request tpo/core/tor!739
This commit is contained in:
David Goulet 2023-08-14 13:00:46 +00:00
commit 2c5c752fd2
2 changed files with 6 additions and 6 deletions

View file

@ -55,7 +55,7 @@ bool hashx_compile_a64(const hashx_program* program, uint8_t* code) {
uint8_t* target = NULL;
int creg = -1;
EMIT(pos, a64_prologue);
for (int i = 0; i < program->code_size; ++i) {
for (size_t i = 0; i < program->code_size; ++i) {
const instruction* instr = &program->code[i];
switch (instr->opcode)
{

View file

@ -678,7 +678,7 @@ bool hashx_program_generate(const siphash_state* key, hashx_program* program) {
/* Calculate ASIC latency:
Assumes 1 cycle latency for all operations and unlimited parallelization. */
for (int i = 0; i < program->code_size; ++i) {
for (size_t i = 0; i < program->code_size; ++i) {
instruction* instr = &program->code[i];
if (instr->dst < 0)
continue;
@ -724,8 +724,8 @@ bool hashx_program_generate(const siphash_state* key, hashx_program* program) {
static const char* x86_reg_map[] = { "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15" };
void hashx_program_asm_x86(const hashx_program* program) {
int target = 0;
for (unsigned i = 0; i < program->code_size; ++i) {
size_t target = 0;
for (size_t i = 0; i < program->code_size; ++i) {
const instruction* instr = &program->code[i];
switch (instr->opcode)
{
@ -762,13 +762,13 @@ void hashx_program_asm_x86(const hashx_program* program) {
break;
case INSTR_TARGET:
printf("test edi, edi\n");
printf("target_%i: cmovz esi, edi\n", i);
printf("target_%i: cmovz esi, edi\n", (int)i);
target = i;
break;
case INSTR_BRANCH:
printf("or edx, esi\n");
printf("test edx, %i\n", instr->imm32);
printf("jz target_%i\n", target);
printf("jz target_%i\n", (int)target);
break;
default:
UNREACHABLE;