diff options
| author | Sergey Andreenko <seandree@microsoft.com> | 2018-09-28 18:57:29 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-09-28 18:57:29 -0700 |
| commit | f7f0929a468f9a4d115b51da0dc5b9328413352c (patch) | |
| tree | eea57b66743373d536631e529dcaf3cb5826fc6b /src/jit/valuenum.cpp | |
| parent | 7d14e3a60dd9eff95e60b919cf30fd9f306e0e75 (diff) | |
| download | coreclr-f7f0929a468f9a4d115b51da0dc5b9328413352c.tar.gz coreclr-f7f0929a468f9a4d115b51da0dc5b9328413352c.tar.bz2 coreclr-f7f0929a468f9a4d115b51da0dc5b9328413352c.zip | |
Make `structType` optional in jitEEInterface method `getFieldType`. (#20191)
* Make `structType` optional in `getFieldType`.
The declaration in corinfo.h says: "if 'structType' == 0, then don't bother the structure info". However, `getFieldTypeInternal ` did not check this case.
* Do not bother the structure info when we do not need it from `getFieldType`.
Diffstat (limited to 'src/jit/valuenum.cpp')
| -rw-r--r-- | src/jit/valuenum.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/jit/valuenum.cpp b/src/jit/valuenum.cpp index 74ff493238..c1559e98da 100644 --- a/src/jit/valuenum.cpp +++ b/src/jit/valuenum.cpp @@ -3493,10 +3493,9 @@ ValueNum ValueNumStore::VNApplySelectorsAssign( } // Otherwise, fldHnd is a real field handle. - CORINFO_FIELD_HANDLE fldHnd = fieldSeq->m_fieldHnd; - CORINFO_CLASS_HANDLE structType = nullptr; + CORINFO_FIELD_HANDLE fldHnd = fieldSeq->m_fieldHnd; noway_assert(fldHnd != nullptr); - CorInfoType fieldCit = m_pComp->info.compCompHnd->getFieldType(fldHnd, &structType); + CorInfoType fieldCit = m_pComp->info.compCompHnd->getFieldType(fldHnd); var_types fieldType = JITtype2varType(fieldCit); ValueNum fieldHndVN = VNForHandle(ssize_t(fldHnd), GTF_ICON_FIELD_HDL); |