diff options
author | Michelle McDaniel <adiaaida@gmail.com> | 2016-06-24 14:10:52 -0700 |
---|---|---|
committer | Michelle McDaniel <adiaaida@gmail.com> | 2016-07-29 09:16:39 -0700 |
commit | a1fa8c14cae981369dccf2f361b01959b215ad8f (patch) | |
tree | cfbd2c5020bacd9422a64338f513350057e3cae0 /src/jit/ee_il_dll.cpp | |
parent | a812669c5737a336745f42c099a1a8a6e1aafa4f (diff) | |
download | coreclr-a1fa8c14cae981369dccf2f361b01959b215ad8f.tar.gz coreclr-a1fa8c14cae981369dccf2f361b01959b215ad8f.tar.bz2 coreclr-a1fa8c14cae981369dccf2f361b01959b215ad8f.zip |
Massage code for clang-format
This change starts the process of updating the jit code to make it ready
for being formatted by clang-format. Changes mostly include reflowing
comments that go past our column limit and moving comments around ifdefs
so clang-format does not modify the indentation. Additionally, some
header files are manually reformatted for pointer alignment and marked
as clang-format off so that we do not lose the current formatting.
Diffstat (limited to 'src/jit/ee_il_dll.cpp')
-rwxr-xr-x | src/jit/ee_il_dll.cpp | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/jit/ee_il_dll.cpp b/src/jit/ee_il_dll.cpp index c726856b9b..d7293d53ba 100755 --- a/src/jit/ee_il_dll.cpp +++ b/src/jit/ee_il_dll.cpp @@ -435,6 +435,7 @@ unsigned Compiler::eeGetArgSize(CORINFO_ARG_LIST_HANDLE list, CORINFO_ // Everything fits into a single 'slot' size // to accommodate irregular sized structs, they are passed byref + CLANG_FORMAT_COMMENT_ANCHOR; #ifdef FEATURE_UNIX_AMD64_STRUCT_PASSING CORINFO_CLASS_HANDLE argClass; @@ -461,9 +462,11 @@ unsigned Compiler::eeGetArgSize(CORINFO_ARG_LIST_HANDLE list, CORINFO_ // make certain the EE passes us back the right thing for refanys assert(argTypeJit != CORINFO_TYPE_REFANY || structSize == 2*sizeof(void*)); -#if FEATURE_MULTIREG_ARGS // For each target that supports passing struct args in multiple registers // apply the target specific rules for them here: + CLANG_FORMAT_COMMENT_ANCHOR; + +#if FEATURE_MULTIREG_ARGS #if defined(_TARGET_ARM64_) // Any structs that are larger than MAX_PASS_MULTIREG_BYTES are always passed by reference if (structSize > MAX_PASS_MULTIREG_BYTES) @@ -484,12 +487,11 @@ unsigned Compiler::eeGetArgSize(CORINFO_ARG_LIST_HANDLE list, CORINFO_ return TARGET_POINTER_SIZE; } } + // otherwise will we pass this struct by value in multiple registers } - // otherwise will we pass this struct by value in multiple registers - // #elif defined(_TARGET_ARM_) // otherwise will we pass this struct by value in multiple registers -#else // +#else NYI("unknown target"); #endif // defined(_TARGET_XXX_) #endif // FEATURE_MULTIREG_ARGS @@ -1269,10 +1271,10 @@ LONG TryResolveTokenFilter(struct _EXCEPTION_POINTERS* exceptionPointers, void* { assert(exceptionPointers->ExceptionRecord->ExceptionCode != SEH_VERIFICATION_EXCEPTION); - // Backward compatibility: Convert bad image format exceptions thrown by the EE while resolving token to verification exceptions - // if we are verifying. Verification exceptions will cause the JIT of the basic block to fail, but the JITing of the whole method - // is still going to succeed. This is done for backward compatibility only. Ideally, we would always treat bad tokens in the IL - // stream as fatal errors. + // Backward compatibility: Convert bad image format exceptions thrown by the EE while resolving token to + // verification exceptions if we are verifying. Verification exceptions will cause the JIT of the basic block to + // fail, but the JITing of the whole method is still going to succeed. This is done for backward compatibility only. + // Ideally, we would always treat bad tokens in the IL stream as fatal errors. if (exceptionPointers->ExceptionRecord->ExceptionCode == EXCEPTION_COMPLUS) { auto* param = reinterpret_cast<TryResolveTokenFilterParam*>(theParam); |