[PATCH] D21107: Generate codeview for array types
David Majnemer via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 7 20:57:01 PDT 2016
majnemer added inline comments.
================
Comment at: lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:785
@@ +784,3 @@
+ TypeIndex IndexType = Asm->MAI->getPointerSize() == 8 ?
+ TypeIndex(SimpleTypeKind::UInt64) :
+ TypeIndex(SimpleTypeKind::UInt32);
----------------
amccarth wrote:
> majnemer wrote:
> > They don't use UInt64Quad? Surprising.
> That's an excellent question. Let me check it out.
x64 MSVC 2015 Update 2 says:
Index type = T_UQUAD(0023)
x86 MSVC 2015 Update 2 says:
Index type = T_ULONG(0022)
Sounds like we should use UInt32Long and UInt64Quad.
http://reviews.llvm.org/D21107
More information about the llvm-commits
mailing list