[PATCH] D157356: [BOLT][DWARF] Always use new low_pc for call_site

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 7 20:33:05 PDT 2023


ayermolo created this revision.
Herald added subscribers: treapster, hoy, modimo, wenlei.
Herald added a reviewer: rafauler.
Herald added a reviewer: Amir.
Herald added a reviewer: maksfb.
Herald added a project: All.
ayermolo requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, jplehr, yota9, sstefan1.
Herald added a project: LLVM.

Changed to creating a new index all the time. This code was legacy of when we
couldn't change the size of .debug_info, and led to subtle bugs where index for
new entries was pointing to a wrong address.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D157356

Files:
  bolt/lib/Rewrite/DWARFRewriter.cpp
  bolt/test/X86/Inputs/dwarf4-df-call-site-change-low-pc-helper.s
  bolt/test/X86/Inputs/dwarf4-df-call-site-change-low-pc-main.s
  bolt/test/X86/Inputs/dwarf5-df-call-site-change-low-pc-helper.s
  bolt/test/X86/Inputs/dwarf5-df-call-site-change-low-pc-main.s
  bolt/test/X86/dwarf4-df-call-site-change-low-pc.test
  bolt/test/X86/dwarf5-df-call-site-change-low-pc.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157356.548048.patch
Type: text/x-patch
Size: 91841 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230808/9b695a2a/attachment.bin>


More information about the llvm-commits mailing list