[all-commits] [llvm/llvm-project] 9bd3ff: Fix -u option in dsymutil, to not emit an extra DW...
Shubham Sandeep Rastogi via All-commits
all-commits at lists.llvm.org
Tue May 30 21:37:59 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 9bd3ff8bf9e05e7bf95a4a068aa3b50813a93975
https://github.com/llvm/llvm-project/commit/9bd3ff8bf9e05e7bf95a4a068aa3b50813a93975
Author: Shubham Sandeep Rastogi <srastogi22 at apple.com>
Date: 2023-05-30 (Tue, 30 May 2023)
Changed paths:
M llvm/lib/DWARFLinker/DWARFLinker.cpp
A llvm/test/tools/dsymutil/ARM/fat-dylib-update.test
Log Message:
-----------
Fix -u option in dsymutil, to not emit an extra DW_LNE_set_address if the original line table was empty
With dsymutil's -u option, only the accelerator tables should be
updated, but with https://reviews.llvm.org/D150554 the -u option will
still re-generate the line table. If the line table was empty, that is,
it was a dummy line table, with no entries in it, dsymutil will always
generate a line table with a DW_LNE_end_sequence, a funky side effect of
this is that when the line table is re-generated, it will always emit a
DW_LNE_set_address first, which will change the line table total size.
This patch addresses this by making sure that if all the line table has
in it is a DW_LNE_end_sequence, it is the same as a dummy entry.
Differential Revision: https://reviews.llvm.org/D151579
More information about the All-commits
mailing list