FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-101278: Drop -gdwarf-4 flag from the BOLT build by corona10 · Pull Request #101280 · python/cpython · GitHub

/ cpython Public

gh-101278: Drop -gdwarf-4 flag from the BOLT build - #101280

Merged
corona10 merged 1 commit into
python:mainfrom
corona10:gh-101278
Jan 24, 2023
Merged

gh-101278: Drop -gdwarf-4 flag from the BOLT build#101280
corona10 merged 1 commit into
python:mainfrom
corona10:gh-101278

Conversation

corona10 commented Jan 24, 2023
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

corona10 commented Jan 24, 2023
edited
Loading

Copy link
Copy Markdown
Member Author

@erlend-aasland @aaupov cc @indygreg
I verified the build success of BOLT with DWARF-5

$ llvm-dwarfdump -r 1 ./python | head | grep Compile
0x00000000: Compile Unit: length = 0x00000102, format = DWARF32, version = 0x0005, unit_type = DW_UT_compile, abbr_offset = 0x0000, addr_size = 0x08 (next unit at 0x00000106)

corona10 merged commit a958e7d into python:main Jan 24, 2023
corona10 deleted the gh-101278 branch January 24, 2023 10:22

aaupov commented Jan 24, 2023
edited
Loading

Copy link
Copy Markdown

BTW, -gdwarf-4 forces the emission of debug information in DWARF4 format. Clang has added an extra flag -fdebug-default-version to specify which version is the default without forcing the output of debug information (https://reviews.llvm.org/D69822).

What's the intended effect here: emit the debug information in default format (-g) or don't emit it at all (removed flag)?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL