VirtualBox

Changeset 98800 in vbox


Ignore:
Timestamp:
Mar 1, 2023 12:42:12 AM (21 months ago)
Author:
vboxsync
Message:

VMM/IEM: Fixed unreachable code issues in IEMAllInstructionsOneByte.cpp.h. bugref:10368

Location:
trunk/src/VBox/VMM/VMMAll
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/VMM/VMMAll/IEMAllInstructionsInterpretOnly.cpp

    r98799 r98800  
    105105#include "IEMAllInstructionsTwoByte0f.cpp.h"
    106106
    107 
    108107#ifdef IEM_WITH_VEX
    109108# include "IEMAllInstructionsVexMap1.cpp.h"
    … …  
    112111#endif
    113112
    114 #ifdef _MSC_VER
    115 # pragma warning(push)
    116 # pragma warning(disable: 4702) /* Unreachable code like return after IEM_MC_ADVANCE_RIP_AND_FINISH. */
    117 #endif
     113#include "IEMAllInstructionsOneByte.cpp.h"
    118114
    119 #include "IEMAllInstructionsOneByte.cpp.h"
    120 #ifdef _MSC_VER
    121 # pragma warning(pop)
    122 #endif
    123 

  • trunk/src/VBox/VMM/VMMAll/IEMAllInstructionsOneByte.cpp.h

    r98103 r98800  
    11001100            IEM_MC_ADVANCE_RIP_AND_FINISH();
    11011101            IEM_MC_END();
    1102             return VINF_SUCCESS;
     1102            break;
    11031103
    11041104        case IEMMODE_32BIT:
    … …  
    11121112            IEM_MC_ADVANCE_RIP_AND_FINISH();
    11131113            IEM_MC_END();
    1114             return VINF_SUCCESS;
     1114            break;
    11151115
    11161116        case IEMMODE_64BIT:
    … …  
    11231123            IEM_MC_ADVANCE_RIP_AND_FINISH();
    11241124            IEM_MC_END();
    1125             return VINF_SUCCESS;
     1125            break;
    11261126
    11271127        IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    22052205            IEM_MC_ADVANCE_RIP_AND_FINISH();
    22062206            IEM_MC_END();
    2207             return VINF_SUCCESS;
     2207            break;
    22082208        }
    22092209
    … …  
    22162216            IEM_MC_ADVANCE_RIP_AND_FINISH();
    22172217            IEM_MC_END();
    2218             return VINF_SUCCESS;
     2218            break;
    22192219        }
    22202220
    … …  
    22272227            IEM_MC_ADVANCE_RIP_AND_FINISH();
    22282228            IEM_MC_END();
    2229             return VINF_SUCCESS;
     2229            break;
    22302230        }
    22312231
    … …  
    22952295                IEM_MC_END();
    22962296            }
    2297             return VINF_SUCCESS;
     2297            break;
    22982298        }
    22992299
    … …  
    23462346                IEM_MC_END();
    23472347            }
    2348             return VINF_SUCCESS;
     2348            break;
    23492349        }
    23502350
    … …  
    23972397                IEM_MC_END();
    23982398            }
    2399             return VINF_SUCCESS;
     2399            break;
    24002400        }
    24012401
    … …  
    24162416    IEMOP_HLP_DEFAULT_64BIT_OP_SIZE();
    24172417
    2418     IEM_MC_BEGIN(0,0);
    24192418    switch (pVCpu->iem.s.enmEffOpSize)
    24202419    {
    24212420        case IEMMODE_16BIT:
     2421            IEM_MC_BEGIN(0,0);
    24222422            IEM_MC_PUSH_U16(i8Imm);
     2423            IEM_MC_ADVANCE_RIP_AND_FINISH();
     2424            IEM_MC_END();
    24232425            break;
    24242426        case IEMMODE_32BIT:
     2427            IEM_MC_BEGIN(0,0);
    24252428            IEM_MC_PUSH_U32(i8Imm);
     2429            IEM_MC_ADVANCE_RIP_AND_FINISH();
     2430            IEM_MC_END();
    24262431            break;
    24272432        case IEMMODE_64BIT:
     2433            IEM_MC_BEGIN(0,0);
    24282434            IEM_MC_PUSH_U64(i8Imm);
     2435            IEM_MC_ADVANCE_RIP_AND_FINISH();
     2436            IEM_MC_END();
    24292437            break;
    24302438        IEM_NOT_REACHED_DEFAULT_CASE_RET();
    24312439    }
    2432     IEM_MC_ADVANCE_RIP_AND_FINISH();
    2433     IEM_MC_END();
    24342440}
    24352441
    … …  
    24942500                IEM_MC_END();
    24952501            }
    2496             return VINF_SUCCESS;
     2502            break;
    24972503
    24982504        case IEMMODE_32BIT:
    … …  
    25432549                IEM_MC_END();
    25442550            }
    2545             return VINF_SUCCESS;
     2551            break;
    25462552
    25472553        case IEMMODE_64BIT:
    … …  
    25922598                IEM_MC_END();
    25932599            }
    2594             return VINF_SUCCESS;
     2600            break;
    25952601
    25962602        IEM_NOT_REACHED_DEFAULT_CASE_RET();
    25972603    }
    2598     AssertFailedReturn(VERR_IEM_IPE_8);
    25992604}
    26002605
    … …  
    36823687                IEM_MC_ADVANCE_RIP_AND_FINISH();
    36833688                IEM_MC_END();
    3684                 return VINF_SUCCESS;
     3689                break;
    36853690
    36863691            case IEMMODE_32BIT:
    … …  
    36963701                IEM_MC_ADVANCE_RIP_AND_FINISH();
    36973702                IEM_MC_END();
    3698                 return VINF_SUCCESS;
     3703                break;
    36993704
    37003705            case IEMMODE_64BIT:
    … …  
    37103715                IEM_MC_ADVANCE_RIP_AND_FINISH();
    37113716                IEM_MC_END();
    3712                 return VINF_SUCCESS;
     3717                break;
    37133718
    37143719            IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    37403745                IEM_MC_ADVANCE_RIP_AND_FINISH();
    37413746                IEM_MC_END();
    3742                 return VINF_SUCCESS;
     3747                break;
    37433748
    37443749            case IEMMODE_32BIT:
    … …  
    37603765                IEM_MC_ADVANCE_RIP_AND_FINISH();
    37613766                IEM_MC_END();
    3762                 return VINF_SUCCESS;
     3767                break;
    37633768
    37643769            case IEMMODE_64BIT:
    … …  
    37793784                IEM_MC_ADVANCE_RIP_AND_FINISH();
    37803785                IEM_MC_END();
    3781                 return VINF_SUCCESS;
     3786                break;
    37823787
    37833788            IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    42594264        IEM_MC_END();
    42604265    }
    4261     return VINF_SUCCESS;
    42624266}
    42634267
    … …  
    65696573    IEM_MC_CALL_CIMPL_2(iemCImpl_int, u8Int, enmInt);
    65706574    IEM_MC_END();
    6571     return VINF_SUCCESS;
    65726575}
    65736576
    … …  
    75467549    IEM_MC_CALL_CIMPL_3(iemCImpl_fldenv, enmEffOpSize, iEffSeg, GCPtrEffSrc);
    75477550    IEM_MC_END();
    7548     return VINF_SUCCESS;
    75497551}
    75507552
    … …  
    75647566    IEM_MC_CALL_CIMPL_1(iemCImpl_fldcw, u16Fsw);
    75657567    IEM_MC_END();
    7566     return VINF_SUCCESS;
    75677568}
    75687569
    … …  
    75837584    IEM_MC_CALL_CIMPL_3(iemCImpl_fnstenv, enmEffOpSize, iEffSeg, GCPtrEffDst);
    75847585    IEM_MC_END();
    7585     return VINF_SUCCESS;
    75867586}
    75877587
    … …  
    94419441    IEM_MC_CALL_CIMPL_3(iemCImpl_frstor, enmEffOpSize, iEffSeg, GCPtrEffSrc);
    94429442    IEM_MC_END();
    9443     return VINF_SUCCESS;
    94449443}
    94459444
    … …  
    94609459    IEM_MC_CALL_CIMPL_3(iemCImpl_fnsave, enmEffOpSize, iEffSeg, GCPtrEffDst);
    94619460    IEM_MC_END();
    9462     return VINF_SUCCESS;
    9463 
    94649461}
    94659462
    … …  
    1053010527            IEM_MC_REL_JMP_S16_AND_FINISH(i16Imm);
    1053110528            IEM_MC_END();
    10532             return VINF_SUCCESS;
     10529            break;
    1053310530        }
    1053410531
    … …  
    1054010537            IEM_MC_REL_JMP_S32_AND_FINISH(i32Imm);
    1054110538            IEM_MC_END();
    10542             return VINF_SUCCESS;
     10539            break;
    1054310540        }
    1054410541
    … …  
    1058110578    IEM_MC_REL_JMP_S8_AND_FINISH(i8Imm);
    1058210579    IEM_MC_END();
    10583     return VINF_SUCCESS;
    1058410580}
    1058510581
    … …  
    1145811454                IEM_MC_CALL_CIMPL_1(iemCImpl_call_16, u16Target);
    1145911455                IEM_MC_END()
    11460                 return VINF_SUCCESS;
     11456                break;
    1146111457
    1146211458            case IEMMODE_32BIT:
    … …  
    1146611462                IEM_MC_CALL_CIMPL_1(iemCImpl_call_32, u32Target);
    1146711463                IEM_MC_END()
    11468                 return VINF_SUCCESS;
     11464                break;
    1146911465
    1147011466            case IEMMODE_64BIT:
    … …  
    1147411470                IEM_MC_CALL_CIMPL_1(iemCImpl_call_64, u64Target);
    1147511471                IEM_MC_END()
    11476                 return VINF_SUCCESS;
     11472                break;
    1147711473
    1147811474            IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    1149311489                IEM_MC_CALL_CIMPL_1(iemCImpl_call_16, u16Target);
    1149411490                IEM_MC_END()
    11495                 return VINF_SUCCESS;
     11491                break;
    1149611492
    1149711493            case IEMMODE_32BIT:
    … …  
    1150411500                IEM_MC_CALL_CIMPL_1(iemCImpl_call_32, u32Target);
    1150511501                IEM_MC_END()
    11506                 return VINF_SUCCESS;
     11502                break;
    1150711503
    1150811504            case IEMMODE_64BIT:
    … …  
    1151511511                IEM_MC_CALL_CIMPL_1(iemCImpl_call_64, u64Target);
    1151611512                IEM_MC_END()
    11517                 return VINF_SUCCESS;
     11513                break;
    1151811514
    1151911515            IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    1155411550            IEM_MC_CALL_CIMPL_3(pfnCImpl, u16Sel, offSeg, enmEffOpSize);
    1155511551            IEM_MC_END();
    11556             return VINF_SUCCESS;
     11552            break;
    1155711553
    1155811554        case IEMMODE_32BIT:
    … …  
    1156811564            IEM_MC_CALL_CIMPL_3(pfnCImpl, u16Sel, offSeg, enmEffOpSize);
    1156911565            IEM_MC_END();
    11570             return VINF_SUCCESS;
     11566            break;
    1157111567
    1157211568        case IEMMODE_64BIT:
    … …  
    1158311579            IEM_MC_CALL_CIMPL_3(pfnCImpl, u16Sel, offSeg, enmEffOpSize);
    1158411580            IEM_MC_END();
    11585             return VINF_SUCCESS;
     11581            break;
    1158611582
    1158711583        IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    1162211618                IEM_MC_SET_RIP_U16_AND_FINISH(u16Target);
    1162311619                IEM_MC_END()
    11624                 return VINF_SUCCESS;
     11620                break;
    1162511621
    1162611622            case IEMMODE_32BIT:
    … …  
    1163011626                IEM_MC_SET_RIP_U32_AND_FINISH(u32Target);
    1163111627                IEM_MC_END()
    11632                 return VINF_SUCCESS;
     11628                break;
    1163311629
    1163411630            case IEMMODE_64BIT:
    … …  
    1163811634                IEM_MC_SET_RIP_U64_AND_FINISH(u64Target);
    1163911635                IEM_MC_END()
    11640                 return VINF_SUCCESS;
     11636                break;
    1164111637
    1164211638            IEM_NOT_REACHED_DEFAULT_CASE_RET();
    … …  
    1165711653                IEM_MC_SET_RIP_U16_AND_FINISH(u16Target);
    1165811654                IEM_MC_END()
    11659                 return VINF_SUCCESS;
     11655                break;
    1166011656
    1166111657            case IEMMODE_32BIT:
    … …  
    1166811664                IEM_MC_SET_RIP_U32_AND_FINISH(u32Target);
    1166911665                IEM_MC_END()
    11670                 return VINF_SUCCESS;
     11666                break;
    1167111667
    1167211668            case IEMMODE_64BIT:
    … …  
    1167911675                IEM_MC_SET_RIP_U64_AND_FINISH(u64Target);
    1168011676                IEM_MC_END()
    11681                 return VINF_SUCCESS;
     11677                break;
    1168211678
    1168311679            IEM_NOT_REACHED_DEFAULT_CASE_RET();
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette