Fix JIT TARGET usage (#31740)
authorSteve MacLean <Steve.MacLean@microsoft.com>
Thu, 6 Feb 2020 19:47:33 +0000 (14:47 -0500)
committerGitHub <noreply@github.com>
Thu, 6 Feb 2020 19:47:33 +0000 (14:47 -0500)
* Fix JIT TARGET usage

Fixes _TARGET* & TARGET*_ comments

Fixes TARGET_XARCH4 reference to TARGET_XARCH

Fixes #31372

* Remove no way assert

* Fix JIT formatting

* Use defined() in optcse.cpp

26 files changed:
src/coreclr/src/jit/codegenarmarch.cpp
src/coreclr/src/jit/codegencommon.cpp
src/coreclr/src/jit/codegenxarch.cpp
src/coreclr/src/jit/compiler.cpp
src/coreclr/src/jit/compiler.h
src/coreclr/src/jit/disasm.cpp
src/coreclr/src/jit/disasm.h
src/coreclr/src/jit/ee_il_dll.cpp
src/coreclr/src/jit/emit.h
src/coreclr/src/jit/emitinl.h
src/coreclr/src/jit/emitxarch.cpp
src/coreclr/src/jit/gentree.cpp
src/coreclr/src/jit/importer.cpp
src/coreclr/src/jit/instr.cpp
src/coreclr/src/jit/instrsxarch.h
src/coreclr/src/jit/lclvars.cpp
src/coreclr/src/jit/lower.cpp
src/coreclr/src/jit/lsra.cpp
src/coreclr/src/jit/lsraarmarch.cpp
src/coreclr/src/jit/morph.cpp
src/coreclr/src/jit/optcse.cpp
src/coreclr/src/jit/reglist.h
src/coreclr/src/jit/simd.cpp
src/coreclr/src/jit/target.h
src/coreclr/src/jit/unwind.cpp
src/coreclr/src/jit/utils.cpp

index ab0e11f..a15916e 100644 (file)
@@ -763,7 +763,7 @@ void CodeGen::genPutArgStk(GenTreePutArgStk* treeNode)
 #else  // !TARGET_ARM64
             // There is no zero register on ARM32
             unreached();
-#endif // !_TARGET_ARM64
+#endif // !TARGET_ARM64
         }
         else
         {
@@ -1344,7 +1344,7 @@ void CodeGen::genMultiRegCallStoreToLocal(GenTree* treeNode)
 #elif defined(TARGET_ARM64)
     // Structs of size >=9 and <=16 are returned in two return registers on ARM64 and HFAs.
     assert(varTypeIsStruct(treeNode));
-#endif // _TARGET_*
+#endif // TARGET*
 
     // Assumption: current implementation requires that a multi-reg
     // var in 'var = call' is flagged as lvIsMultiRegRet to prevent it from
@@ -2496,7 +2496,7 @@ void CodeGen::genCallInstruction(GenTreeCall* call)
         GetEmitter()->emitIns_R_R_I(INS_ldr, EA_4BYTE, tmpReg, regThis, 0);
 #elif defined(TARGET_ARM64)
         GetEmitter()->emitIns_R_R_I(INS_ldr, EA_4BYTE, REG_ZR, regThis, 0);
-#endif // _TARGET_*
+#endif // TARGET*
     }
 
     // Either gtControlExpr != null or gtCallAddr != null or it is a direct non-virtual call to a user or helper
@@ -3334,7 +3334,7 @@ void CodeGen::genFloatToFloatCast(GenTree* treeNode)
         GetEmitter()->emitIns_R_R(INS_mov, emitActualTypeSize(treeNode), treeNode->GetRegNum(), op1->GetRegNum());
     }
 
-#endif // _TARGET_*
+#endif // TARGET*
 
     genProduceReg(treeNode);
 }
index 92f8fed..4276c50 100644 (file)
@@ -243,8 +243,8 @@ int CodeGenInterface::genCallerSPtoFPdelta() const
     //     pushed ebp
     callerSPtoFPdelta -= 2 * REGSIZE_BYTES;
 #else
-#error "Unknown _TARGET_"
-#endif // _TARGET_*
+#error "Unknown TARGET"
+#endif // TARGET*
 
     assert(callerSPtoFPdelta <= 0);
     return callerSPtoFPdelta;
@@ -273,8 +273,8 @@ int CodeGenInterface::genCallerSPtoInitialSPdelta() const
         callerSPtoSPdelta -= REGSIZE_BYTES;
     }
 #else
-#error "Unknown _TARGET_"
-#endif // _TARGET_*
+#error "Unknown TARGET"
+#endif // TARGET*
 
     assert(callerSPtoSPdelta <= 0);
     return callerSPtoSPdelta;
@@ -1509,7 +1509,7 @@ AGAIN:
             break;
 
 #endif // SCALED_ADDR_MODES
-#endif // !_TARGET_ARMARCH
+#endif // !TARGET_ARMARCH
 
         case GT_NOP:
 
@@ -1587,7 +1587,7 @@ AGAIN:
             break;
 
 #endif // SCALED_ADDR_MODES
-#endif // !_TARGET_ARMARCH
+#endif // !TARGET_ARMARCH
 
         case GT_NOP:
 
@@ -5383,7 +5383,7 @@ void          CodeGen::genPushCalleeSavedRegisters()
 
 #else
     assert(!"Unknown TARGET");
-#endif // _TARGET_*
+#endif // TARGET*
 }
 
 #if defined(TARGET_ARM)
@@ -5622,7 +5622,7 @@ void CodeGen::genZeroInitFltRegs(const regMaskTP& initFltRegs, const regMaskTP&
 #elif defined(TARGET_ARM64)
                 // We will just zero out the entire vector register. This sets it to a double/float zero value
                 GetEmitter()->emitIns_R_I(INS_movi, EA_16BYTE, reg, 0x00, INS_OPTS_16B);
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
 #endif
                 fltInitReg = reg;
@@ -5657,7 +5657,7 @@ void CodeGen::genZeroInitFltRegs(const regMaskTP& initFltRegs, const regMaskTP&
 #elif defined(TARGET_ARM64)
                 // We will just zero out the entire vector register. This sets it to a double/float zero value
                 GetEmitter()->emitIns_R_I(INS_movi, EA_16BYTE, reg, 0x00, INS_OPTS_16B);
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
 #endif
                 dblInitReg = reg;
@@ -6071,7 +6071,7 @@ void CodeGen::genPopCalleeSavedRegisters(bool jmpEpilog)
     noway_assert(compiler->compCalleeRegsPushed == popCount);
 }
 
-#endif // _TARGET_*
+#endif // TARGET*
 
 // We need a register with value zero. Zero the initReg, if necessary, and set *pInitRegZeroed if so.
 // Return the register to use. On ARM64, we never touch the initReg, and always just return REG_ZR.
@@ -6353,9 +6353,9 @@ void CodeGen::genZeroInitFrame(int untrLclHi, int untrLclLo, regNumber initReg,
         }
 #endif // !UNIX_AMD64_ABI
 
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
     }
     else if (genInitStkLclCnt > 0)
     {
@@ -8249,7 +8249,7 @@ void CodeGen::genFnEpilog(BasicBlock* block)
                 regSet.verifyRegUsed(REG_ECX);
             }
             else
-#endif // _TARGET_X86
+#endif // TARGET_X86
             {
                 /* Add 'compiler->compLclFrameSize' to ESP */
                 /* Generate "add esp, <stack-size>" */
@@ -8319,7 +8319,7 @@ void CodeGen::genFnEpilog(BasicBlock* block)
                 inst_RV(INS_pop, REG_ECX, TYP_I_IMPL);
                 regSet.verifyRegUsed(REG_ECX);
             }
-#endif // _TARGET_X86
+#endif // TARGET_X86
             else
             {
                 // We need to make ESP point to the callee-saved registers
@@ -8547,9 +8547,9 @@ void CodeGen::genFnEpilog(BasicBlock* block)
     }
 }
 
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
 #if defined(FEATURE_EH_FUNCLETS)
 
@@ -9239,7 +9239,7 @@ void CodeGen::genCaptureFuncletPrologEpilogInfo()
     }
 }
 
-#else // _TARGET_*
+#else // TARGET*
 
 /*****************************************************************************
  *
@@ -9274,7 +9274,7 @@ void CodeGen::genCaptureFuncletPrologEpilogInfo()
     }
 }
 
-#endif // _TARGET_*
+#endif // TARGET*
 
 /*-----------------------------------------------------------------------------
  *
@@ -9423,11 +9423,11 @@ void CodeGen::genSetPSPSym(regNumber initReg, bool* pInitRegZeroed)
 
     GetEmitter()->emitIns_S_R(ins_Store(TYP_I_IMPL), EA_PTRSIZE, REG_SPBASE, compiler->lvaPSPSym, 0);
 
-#else // _TARGET_*
+#else // TARGET*
 
     NYI("Set function PSP sym");
 
-#endif // _TARGET_*
+#endif // TARGET*
 }
 
 #endif // FEATURE_EH_FUNCLETS
@@ -9866,7 +9866,7 @@ unsigned CodeGen::getFirstArgWithStackSlot()
     return baseVarNum;
 #elif defined(TARGET_AMD64)
     return 0;
-#else  // _TARGET_X86
+#else  // TARGET_X86
     // Not implemented for x86.
     NYI_X86("getFirstArgWithStackSlot not yet implemented for x86.");
     return BAD_VAR_NUM;
index 977a9e2..4c12d3f 100644 (file)
@@ -7585,7 +7585,7 @@ void CodeGen::genRemoveAlignmentAfterCall(GenTreeCall* call, unsigned bias)
 #endif // !UNIX_X86_ABI_
 #else  // TARGET_X86
     assert(bias == 0);
-#endif // !_TARGET_X86
+#endif // !TARGET_X86
 }
 
 #ifdef TARGET_X86
index f4bc154..6ea252b 100644 (file)
@@ -822,11 +822,11 @@ var_types Compiler::getArgTypeForStruct(CORINFO_CLASS_HANDLE clsHnd,
                 howToPassStruct = SPK_ByValue;
                 useType         = TYP_STRUCT;
 
-#else //  _TARGET_XXX_
+#else //  TARGET_XXX
 
                 noway_assert(!"Unhandled TARGET in getArgTypeForStruct (with FEATURE_MULTIREG_ARGS=1)");
 
-#endif //  _TARGET_XXX_
+#endif //  TARGET_XXX
             }
         }
         else // (structSize > MAX_PASS_MULTIREG_BYTES)
@@ -849,11 +849,11 @@ var_types Compiler::getArgTypeForStruct(CORINFO_CLASS_HANDLE clsHnd,
             howToPassStruct = SPK_ByReference;
             useType         = TYP_UNKNOWN;
 
-#else //  _TARGET_XXX_
+#else //  TARGET_XXX
 
             noway_assert(!"Unhandled TARGET in getArgTypeForStruct");
 
-#endif //  _TARGET_XXX_
+#endif //  TARGET_XXX
         }
     }
 
@@ -1080,11 +1080,11 @@ var_types Compiler::getReturnTypeForStruct(CORINFO_CLASS_HANDLE clsHnd,
                 howToReturnStruct = SPK_ByReference;
                 useType           = TYP_UNKNOWN;
 
-#else //  _TARGET_XXX_
+#else //  TARGET_XXX
 
                 noway_assert(!"Unhandled TARGET in getReturnTypeForStruct (with FEATURE_MULTIREG_ARGS=1)");
 
-#endif //  _TARGET_XXX_
+#endif //  TARGET_XXX
             }
         }
         else // (structSize > MAX_RET_MULTIREG_BYTES) || (FEATURE_MULTIREG_RET == 0)
index 33a1859..c974232 100644 (file)
@@ -1616,7 +1616,7 @@ public:
 
     bool isPassedInFloatRegisters()
     {
-#ifdef _TARGET_X86
+#ifdef TARGET_X86
         return false;
 #else
         return isValidFloatArgReg(GetRegNum());
index 9190820..5896454 100644 (file)
@@ -319,9 +319,9 @@ size_t DisAssembler::disCchAddrMember(
             break;
     }
 
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
     if (retval == 0)
     {
@@ -568,9 +568,9 @@ size_t DisAssembler::disCchFixupMember(
             break;
     }
 
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
     /* no displacement */
 
@@ -781,9 +781,9 @@ size_t DisAssembler::disCchRegRelMember(
             break;
     }
 
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
     /* save displacement */
 
@@ -1083,9 +1083,9 @@ size_t DisAssembler::CbDisassemble(DIS*        pdis,
                 break;
 
         } // end switch
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
         return cb;
     } // end if
@@ -1252,7 +1252,7 @@ void DisAssembler::DisasmBuffer(FILE* pfile, bool printit)
     pdis = DIS::PdisNew(DIS::distX8664);
 #elif defined(TARGET_ARM64)
     pdis = DIS::PdisNew(DIS::distArm64);
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
 #endif
 
index c4d09a3..f7cea20 100644 (file)
@@ -45,7 +45,7 @@ XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
 #include "disx86.h"
 #elif defined(TARGET_ARM64)
 #include "disarm64.h"
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
 #endif
 
index d5d3bbd..cbf8d59 100644 (file)
@@ -517,7 +517,7 @@ unsigned Compiler::eeGetArgSize(CORINFO_ARG_LIST_HANDLE list, CORINFO_SIG_INFO*
 //  otherwise will we pass this struct by value in multiple registers
 #else
         NYI("unknown target");
-#endif // defined(_TARGET_XXX_)
+#endif // defined(TARGET_XXX)
 #endif // FEATURE_MULTIREG_ARGS
 
         // we pass this struct by value in multiple registers
index cec64ec..37a1795 100644 (file)
@@ -1309,7 +1309,7 @@ protected:
 #define PERFSCORE_LATENCY_WR_GENERAL PERFSCORE_LATENCY_1C
 #define PERFSCORE_LATENCY_RD_WR_GENERAL PERFSCORE_LATENCY_4C
 
-#endif // _TARGET_XXX
+#endif // TARGET_XXX
 
 // Make this an enum:
 //
index 6647c61..82ea2e4 100644 (file)
@@ -13,7 +13,7 @@ inline bool emitter::instrIs3opImul(instruction ins)
 {
 #ifdef TARGET_X86
     return ((ins >= INS_imul_AX) && (ins <= INS_imul_DI));
-#else // _TARGET_AMD64
+#else // TARGET_AMD64
     return ((ins >= INS_imul_AX) && (ins <= INS_imul_15));
 #endif
 }
@@ -23,7 +23,7 @@ inline bool emitter::instrIsExtendedReg3opImul(instruction ins)
 {
 #ifdef TARGET_X86
     return false;
-#else // _TARGET_AMD64
+#else // TARGET_AMD64
     return ((ins >= INS_imul_08) && (ins <= INS_imul_15));
 #endif
 }
index ca5b162..22b800e 100644 (file)
@@ -456,7 +456,7 @@ bool TakesRexWPrefix(instruction ins, emitAttr attr)
     {
         return false;
     }
-#else  //!_TARGET_AMD64 = TARGET_X86
+#else  //! TARGET_AMD64 = TARGET_X86
     return false;
 #endif //! TARGET_AMD64
 }
@@ -7952,7 +7952,7 @@ void emitter::emitDispInsHex(instrDesc* id, BYTE* code, size_t sz)
 #ifdef TARGET_AMD64
         // how many bytes per instruction we format for
         const size_t digits = 10;
-#else // _TARGET_X86
+#else // TARGET_X86
         const size_t digits = 6;
 #endif
         printf(" ");
@@ -14283,7 +14283,7 @@ emitter::insExecutionCharacteristics emitter::getInsExecutionCharacteristics(ins
                 result.insLatency = PERFSCORE_LATENCY_1C;
             }
             break;
-#endif // _TARGET_X86
+#endif // TARGET_X86
 
 #ifdef TARGET_AMD64
         case INS_movsq:
index 5e0eb6a..6972c0b 100644 (file)
@@ -3035,7 +3035,7 @@ bool Compiler::gtMarkAddrMode(GenTree* addr, int* pCostEx, int* pCostSz, var_typ
             }
         }
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 
         assert(addr->gtOper == GT_ADD);
@@ -3333,7 +3333,7 @@ unsigned Compiler::gtSetEvalOrder(GenTree* tree)
             case GT_CNS_LNG:
             case GT_CNS_STR:
             case GT_CNS_INT:
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 
             COMMON_CNS:
@@ -3532,7 +3532,7 @@ unsigned Compiler::gtSetEvalOrder(GenTree* tree)
                         costSz = 6;
                     }
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 
                     /* Overflow casts are a lot more expensive */
@@ -16759,7 +16759,7 @@ GenTree* Compiler::gtGetSIMDZero(var_types simdType, var_types baseType, CORINFO
                 }
                 break;
 
-#if defined(_TARGET_XARCH4_) && defined(FEATURE_HW_INTRINSICS)
+#if defined(TARGET_XARCH) && defined(FEATURE_HW_INTRINSICS)
             case TYP_SIMD32:
                 switch (baseType)
                 {
@@ -18490,7 +18490,7 @@ void ReturnTypeDesc::InitializeStructReturnType(Compiler* comp, CORINFO_CLASS_HA
                 m_regType[i] = comp->getJitGCType(gcPtrs[i]);
             }
 
-#else //  _TARGET_XXX_
+#else //  TARGET_XXX
 
             // This target needs support here!
             //
index 8875a29..04c1033 100644 (file)
@@ -5728,7 +5728,7 @@ void Compiler::verVerifyCond(const typeInfo& tiOp1, const typeInfo& tiOp2, unsig
     {
 #ifdef TARGET_64BIT
         Verify(tiCompatibleWith(tiOp1, tiOp2, true), "Cond type mismatch");
-#else  // _TARGET_64BIT
+#else  // TARGET_64BIT
         // [10/17/2013] Consider changing this: to put on my verification lawyer hat,
         // this is non-conforming to the ECMA Spec: types don't have to be equivalent,
         // but compatible, since we can coalesce native int with int32 (see section III.1.5).
@@ -14120,10 +14120,6 @@ void Compiler::impImportBlockCode(BasicBlock* block)
 
                 lclTyp = JITtype2varType(ciType);
 
-#ifdef _TARGET_AMD64
-                noway_assert(varTypeIsIntegralOrI(lclTyp) || varTypeIsFloating(lclTyp) || lclTyp == TYP_STRUCT);
-#endif // _TARGET_AMD64
-
                 if (compIsForInlining())
                 {
                     switch (fieldInfo.fieldAccessor)
index 0b31b8a..52ab884 100644 (file)
@@ -67,7 +67,7 @@ const char* CodeGen::genInsName(instruction ins)
         #include "instrs.h"
 
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
     };
     // clang-format on
@@ -905,7 +905,7 @@ void CodeGen::inst_RV_SH(
 
 #else
     NYI("inst_RV_SH - unknown target");
-#endif // _TARGET_*
+#endif // TARGET*
 }
 
 /*****************************************************************************
@@ -1854,8 +1854,8 @@ instruction CodeGen::ins_Copy(var_types dstType)
     assert(!varTypeIsSIMD(dstType));
     assert(!varTypeIsFloating(dstType));
     return INS_mov;
-#else // _TARGET_*
-#error "Unknown _TARGET_"
+#else // TARGET*
+#error "Unknown TARGET"
 #endif
 }
 
@@ -2316,7 +2316,7 @@ void CodeGen::instGen_MemoryBarrier()
 #elif defined(TARGET_ARM64)
     GetEmitter()->emitIns_BARR(INS_dmb, barrierType);
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 }
 
@@ -2331,7 +2331,7 @@ void CodeGen::instGen_Set_Reg_To_Zero(emitAttr size, regNumber reg, insFlags fla
 #elif defined(TARGET_ARMARCH)
     GetEmitter()->emitIns_R_I(INS_mov, size, reg, 0 ARM_ARG(flags));
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
     regSet.verifyRegUsed(reg);
 }
@@ -2348,7 +2348,7 @@ void CodeGen::instGen_Compare_Reg_To_Zero(emitAttr size, regNumber reg)
 #elif defined(TARGET_ARMARCH)
     GetEmitter()->emitIns_R_I(INS_cmp, size, reg, 0);
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 }
 
@@ -2362,7 +2362,7 @@ void CodeGen::instGen_Compare_Reg_To_Reg(emitAttr size, regNumber reg1, regNumbe
 #if defined(TARGET_XARCH) || defined(TARGET_ARMARCH)
     GetEmitter()->emitIns_R_R(INS_cmp, size, reg1, reg2);
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
 }
 
@@ -2409,7 +2409,7 @@ void CodeGen::instGen_Compare_Reg_To_Imm(emitAttr size, regNumber reg, target_ss
             assert(!"Invalid immediate for instGen_Compare_Reg_To_Imm");
         }
 #else
-#error "Unknown _TARGET_"
+#error "Unknown TARGET"
 #endif
     }
 }
index 0bd795d..986ce9a 100644 (file)
@@ -694,7 +694,7 @@ INST1(shrd,             "shrd",             IUM_RW, 0x0F00AC,
 #ifdef TARGET_X86
 INST1(fld,              "fld",              IUM_WR, 0x0000D9,                                                            INS_FLAGS_x87Instr)
 INST1(fstp,             "fstp",             IUM_WR, 0x0018D9,                                                            INS_FLAGS_x87Instr)
-#endif // _TARGET_X86
+#endif // TARGET_X86
 
 INST1(seto,             "seto",             IUM_WR, 0x0F0090,                                                            INS_FLAGS_ReadsFlags)
 INST1(setno,            "setno",            IUM_WR, 0x0F0091,                                                            INS_FLAGS_ReadsFlags)
index a411073..4a5b2f1 100644 (file)
@@ -557,7 +557,7 @@ void Compiler::lvaInitUserArgs(InitVarDscInfo* varDscInfo)
 #elif defined(TARGET_AMD64) && !defined(UNIX_AMD64_ABI)
     // On System V type environment the float registers are not indexed together with the int ones.
     varDscInfo->floatRegArgNum = varDscInfo->intRegArgNum;
-#endif // _TARGET_*
+#endif // TARGET*
 
     CORINFO_ARG_LIST_HANDLE argLst = info.compMethodInfo->args.args;
 
@@ -1000,7 +1000,7 @@ void Compiler::lvaInitUserArgs(InitVarDscInfo* varDscInfo)
             //
             varDscInfo->setAllRegArgUsed(argType);
 
-#endif // _TARGET_XXX_
+#endif // TARGET_XXX
 
 #if FEATURE_FASTTAILCALL
             varDsc->lvStkOffs = varDscInfo->stackArgSize;
@@ -5370,9 +5370,9 @@ int Compiler::lvaAssignVirtualFrameOffsetToArg(unsigned lclNum,
             varDsc->lvStkOffs = argOffs;
             argOffs += argSize;
         }
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
     }
     else
     {
index b6def42..676ddd2 100644 (file)
@@ -2306,7 +2306,7 @@ GenTree* Lowering::LowerTailCallViaHelper(GenTreeCall* call, GenTree* callTarget
 
 #else
     NYI("LowerTailCallViaHelper");
-#endif // _TARGET_*
+#endif // TARGET*
 
     // Transform this call node into a call to Jit tail call helper.
     call->gtCallType    = CT_HELPER;
index 0949203..f32d7bd 100644 (file)
@@ -1428,7 +1428,7 @@ bool LinearScan::isRegCandidate(LclVarDsc* varDsc)
         // Lowering will have split any candidate lclVars into lo/hi vars.
         return false;
     }
-#endif // !defined(_TARGET_64BIT)
+#endif // !defined(TARGET_64BIT)
 
     // If we have JMP, reg args must be put on the stack
 
index 3ddab72..303b1c7 100644 (file)
@@ -502,7 +502,7 @@ int LinearScan::BuildPutArgSplit(GenTreePutArgSplit* argNode)
                 currentRegCount = node->AsMultiRegOp()->GetRegCount();
             }
             else
-#endif // _TARGET_ARM
+#endif // TARGET_ARM
             {
                 assert(!node->IsMultiRegNode());
                 currentRegCount = 1;
index aed9902..54bbdb0 100644 (file)
@@ -2975,7 +2975,7 @@ void Compiler::fgInitArgInfo(GenTreeCall* call)
 
 #else
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
 
         bool      isBackFilled     = false;
         unsigned  nextFltArgRegNum = fltArgRegNum; // This is the next floating-point argument register number to use
@@ -3079,7 +3079,7 @@ void Compiler::fgInitArgInfo(GenTreeCall* call)
         }
 #else
 #error Unsupported or unset target architecture
-#endif // _TARGET_XXX_
+#endif // TARGET_XXX
         if (isStructArg)
         {
             // We have an argument with a struct type, but it may be be a child of a GT_COMMA
@@ -11185,7 +11185,7 @@ GenTree* Compiler::fgMorphSmpOp(GenTree* tree, MorphAddrContext* mac)
 //
 // Note for TARGET_ARMARCH we don't have  a remainder instruction, so we don't do this optimization
 //
-#else  // _TARGET_XARCH
+#else  // TARGET_XARCH
             /* If this is an unsigned long mod with op2 which is a cast to long from a
                constant int, then don't morph to a call to the helper.  This can be done
                faster inline using idiv.
@@ -11224,7 +11224,7 @@ GenTree* Compiler::fgMorphSmpOp(GenTree* tree, MorphAddrContext* mac)
                     return tree;
                 }
             }
-#endif // _TARGET_XARCH
+#endif // TARGET_XARCH
 
         ASSIGN_HELPER_FOR_MOD:
 
index cb106f4..4a9a405 100644 (file)
@@ -1664,7 +1664,7 @@ public:
                 largeFrame = true;
                 break; // early out,  we don't need to keep increasing frameSize
             }
-#elif _TARGET_ARM32
+#elif defined(TARGET_ARM)
             if (frameSize > 0x0400)
             {
                 // We likely have a large stack frame.
@@ -1679,7 +1679,7 @@ public:
                 hugeFrame = true;
                 break; // early out,  we don't need to keep increasing frameSize
             }
-#elif TARGET_ARM64
+#elif defined(TARGET_ARM64)
             if (frameSize > 0x1000)
             {
                 // We likely have a large stack frame.
index 58b02b0..16239b8 100644 (file)
@@ -13,6 +13,6 @@
 typedef TinyArray<unsigned short, regNumber, REGNUM_BITS> regList;
 #else
 // The regList is unused for all other targets.
-#endif // _TARGET_*
+#endif // TARGET*
 
 #endif // REGLIST_H
index 6a25c9e..1d7b9bf 100644 (file)
@@ -438,7 +438,7 @@ var_types Compiler::getBaseTypeAndSizeOfSIMDType(CORINFO_CLASS_HANDLE typeHnd, u
             JITDUMP("  Known type Vector256<ulong>\n");
         }
         else
-#endif // defined(_TARGET_XARCH)
+#endif // defined(TARGET_XARCH)
             if (typeHnd == m_simdHandleCache->Vector128FloatHandle)
         {
             simdBaseType = TYP_FLOAT;
@@ -1763,7 +1763,7 @@ GenTree* Compiler::impSIMDAbs(CORINFO_CLASS_HANDLE typeHnd, var_types baseType,
     {
         retVal = gtNewSIMDNode(simdType, op1, SIMDIntrinsicAbs, baseType, size);
     }
-#else // !defined(_TARGET_XARCH)_ && !defined(TARGET_ARM64)
+#else // !defined(TARGET_XARCH)_ && !defined(TARGET_ARM64)
     assert(!"Abs intrinsic on non-xarch target not implemented");
 #endif // !TARGET_XARCH
 
index cab9272..117a3e6 100644 (file)
@@ -3,8 +3,8 @@
 // See the LICENSE file in the project root for more information.
 
 /*****************************************************************************/
-#ifndef _TARGET_H_
-#define _TARGET_H_
+#ifndef TARGET_H_
+#define TARGET_H_
 
 #if defined(FEATURE_CORECLR) && defined(TARGET_UNIX)
 #define FEATURE_VARARG 0
@@ -1986,5 +1986,5 @@ C_ASSERT(sizeof(target_size_t) == TARGET_POINTER_SIZE);
 C_ASSERT(sizeof(target_ssize_t) == TARGET_POINTER_SIZE);
 
 /*****************************************************************************/
-#endif // _TARGET_H_
+#endif // TARGET_H_
 /*****************************************************************************/
index 96c8a98..dff2df6 100644 (file)
@@ -426,8 +426,8 @@ UNATIVE_OFFSET Compiler::unwindGetCurrentOffset(FuncInfoDsc* func)
 
 // See unwindX86.cpp
 
-#else // _TARGET_*
+#else // TARGET*
 
 #error Unsupported or unset target architecture
 
-#endif // _TARGET_*
+#endif // TARGET*
index 7ac1513..d493c99 100644 (file)
@@ -336,9 +336,9 @@ void dspRegMask(regMaskTP regMask, size_t minSiz)
                 }
 #elif defined(TARGET_X86)
 // No register ranges
-#else // _TARGET_*
+#else // TARGET*
 #error Unsupported or unset target architecture
-#endif // _TARGET_*
+#endif // TARGET*
             }
 
 #if defined(TARGET_ARM64)