[PATCH] D127613: New test case to make sure DW_AT_high_pc convert to DW_AT_ranges correctly in Dwarf 3

[Alexander Yermolovich via Phabricator via llvm-commits]

ayermolo added inline comments.


================
Comment at: bolt/test/X86/dwarf3-lowpc-highpc-convert.s:17
+
+# POSTCHECK: [[#%.8x,IDX:]] [[#%.16x, ADDR_1_BEGIN:]]
+# POSTCHECK-SAME: [[#%.16x, ADDR_1_END:]]
----------------
It's not IDX it's an offset, so maybe different name?


================
Comment at: bolt/test/X86/dwarf3-lowpc-highpc-convert.s:53
+	.file	"main.cpp"
+	.file	1 "/home/zr33/test" "main.cpp"
+	.globl	_Z3usePiS_                      # -- Begin function _Z3usePiS_
----------------
Modify path to remove your username...


================
Comment at: bolt/test/X86/dwarf3-lowpc-highpc-convert.s:399
+.Linfo_string0:
+	.asciz	"clang version 14.0.0 (ssh://git.vip.facebook.com/data/gitrepos/osmeta/external/llvm-project d05778fb02f61407d193b874faea6b9521d69361)" # string offset=0
+.Linfo_string1:
----------------
cleanup the string


================
Comment at: bolt/test/X86/dwarf3-lowpc-highpc-convert.s:422
+	.asciz	"char"                          # string offset=197
+	.ident	"clang version 14.0.0 (ssh://git.vip.facebook.com/data/gitrepos/osmeta/external/llvm-project d05778fb02f61407d193b874faea6b9521d69361)"
+	.section	".note.GNU-stack","", at progbits
----------------
ditto


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127613/new/

https://reviews.llvm.org/D127613