rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm with minor comment adjustments
================
Comment at: llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:3167
+
+ // TODO: Need to support bigger ints like __int128.
+ OS.AddComment("Value");
----------------
I don't think we need a TODO here. If Microsoft ever adds an encoding for large
integers, the code change will be elsewhere, and in the process of testing,
this size 10 array is bounds checked during the writing process, so we don't
need this comment to remind us to change this code.
================
Comment at: llvm/test/DebugInfo/COFF/integer-128.ll:22
+; ASM-NEXT: .long 4110 # Type
+; ASM-NEXT: .byte 0x0a, 0x80, 0xff, 0xff # Value
+; ASM-NEXT: .byte 0xff, 0xff, 0xff, 0xff
----------------
(no AI) I see, in the future we should make this use the "streaming" output
style so this comes out as `.short .quad` directives.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105320/new/
https://reviews.llvm.org/D105320
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
