Fix few gcc build warnings for RISC-V (dotnet#102426)

Author: gbalykov

src/coreclr/debug/ee/controller.cpp

@@ -6058,6 +6058,8 @@ bool DebuggerStepper::TrapStep(ControllerStackInfo *info, bool in)
         }
     }
     LOG((LF_CORDB,LL_INFO1000,"Ending TrapStep\n"));
+
+    return false;
 }
 
 bool DebuggerStepper::IsAddrWithinFrame(DebuggerJitInfo *dji,

src/coreclr/jit/codegenriscv64.cpp

@@ -1943,7 +1943,7 @@ void CodeGen::genLclHeap(GenTree* tree)
     noway_assert(isFramePointerUsed()); // localloc requires Frame Pointer to be established since SP changes
     noway_assert(genStackLevel == 0);   // Can't have anything on the stack
 
-    const target_ssize_t pageSize = compiler->eeGetPageSize();
+    const target_size_t pageSize = compiler->eeGetPageSize();
 
     // According to RISC-V Privileged ISA page size is 4KiB
     noway_assert(pageSize == 0x1000);

src/coreclr/vm/riscv64/asmhelpers.S

@@ -382,7 +382,7 @@ NESTED_ENTRY ThePreStub, _TEXT, NoHandler
     EPILOG_BRANCH_REG  t4
 NESTED_END ThePreStub, _TEXT
 
-// ------------------------------------------------------------------\
+// ------------------------------------------------------------------
 
 // EXTERN_C int __fastcall HelperMethodFrameRestoreState(
 // INDEBUG_COMMA(HelperMethodFrame *pFrame)

src/coreclr/vm/riscv64/profiler.cpp

@@ -112,10 +112,10 @@ LPVOID ProfileArgIterator::CopyStructFromRegisters(const ArgLocDesc* sir)
     PROFILE_PLATFORM_SPECIFIC_DATA* pData = reinterpret_cast<PROFILE_PLATFORM_SPECIFIC_DATA*>(m_handle);
 
     struct { bool isFloat, is8; } fields[] = {
-        { sir->m_structFields & (STRUCT_FLOAT_FIELD_FIRST | STRUCT_FLOAT_FIELD_ONLY_TWO | STRUCT_FLOAT_FIELD_ONLY_ONE),
-          sir->m_structFields & STRUCT_FIRST_FIELD_SIZE_IS8 },
-        { sir->m_structFields & (STRUCT_FLOAT_FIELD_SECOND | STRUCT_FLOAT_FIELD_ONLY_TWO),
-          sir->m_structFields & STRUCT_SECOND_FIELD_SIZE_IS8 },
+        { (bool) (sir->m_structFields & (STRUCT_FLOAT_FIELD_FIRST | STRUCT_FLOAT_FIELD_ONLY_TWO | STRUCT_FLOAT_FIELD_ONLY_ONE)),
+          (bool) (sir->m_structFields & STRUCT_FIRST_FIELD_SIZE_IS8) },
+        { (bool) (sir->m_structFields & (STRUCT_FLOAT_FIELD_SECOND | STRUCT_FLOAT_FIELD_ONLY_TWO)),
+          (bool) (sir->m_structFields & STRUCT_SECOND_FIELD_SIZE_IS8) },
     };
     int fieldCount = (sir->m_structFields & STRUCT_FLOAT_FIELD_ONLY_ONE) ? 1 : 2;
     UINT64 bufferPosBegin = m_bufferPos;
@@ -191,11 +191,11 @@ LPVOID ProfileArgIterator::GetNextArgAddr()
         }
     }
 
-    int argSize = m_argIterator.IsArgPassedByRef() ? sizeof(void*) : m_argIterator.GetArgSize();
+    int argSize = m_argIterator.IsArgPassedByRef() ? (int)sizeof(void*) : m_argIterator.GetArgSize();
     if (TransitionBlock::IsFloatArgumentRegisterOffset(argOffset))
     {
         int offset = argOffset - TransitionBlock::GetOffsetOfFloatArgumentRegisters();
-        _ASSERTE(offset + argSize <= sizeof(pData->floatArgumentRegisters));
+        _ASSERTE(offset + argSize <= (int)sizeof(pData->floatArgumentRegisters));
         return (LPBYTE)&pData->floatArgumentRegisters + offset;
     }
 
@@ -204,7 +204,7 @@ LPVOID ProfileArgIterator::GetNextArgAddr()
     if (TransitionBlock::IsArgumentRegisterOffset(argOffset))
     {
         int offset = argOffset - TransitionBlock::GetOffsetOfArgumentRegisters();
-        if (offset + argSize > sizeof(pData->argumentRegisters))
+        if (offset + argSize > (int)sizeof(pData->argumentRegisters))
         {
             // Struct partially spilled on stack
             const int regIndex = NUM_ARGUMENT_REGISTERS - 1;  // first part of struct must be in last register

src/coreclr/vm/riscv64/stubs.cpp

@@ -1070,7 +1070,7 @@ void StubLinkerCPU::EmitMovConstant(IntReg reg, UINT64 imm)
     if (high31 & 0x800)
     {
         // EmitAddImm does not allow negative immediate values, so use EmitSubImm.
-        EmitSubImm(reg, reg, ~high31 + 1 & 0xFFF);
+        EmitSubImm(reg, reg, (~high31 + 1) & 0xFFF);
     }
     else
     {