Skip to content

Commit

Permalink
Rearrange default_configuration and CheckCfg::fill_well_known.
Browse files Browse the repository at this point in the history
There are comments saying these two functions should be kept in sync,
but they have very different structures, process symbols in different
orders, and there are some inconsistencies.

This commit reorders them so they're both mostly processing symbols in
alphabetical order, which makes cross-checking them a lot easier. The
commit also adds some macros to factor out repetitive code patterns.
Plus it adds `sanitizer_cfi_normalize_{integers,pointers}` to
`fill_well_known`, which were missing.

The commit also moves the handling of `sym::test` out of
`build_configuration` into `default_configuration`, where all the other
symbols are handled.
  • Loading branch information
nnethercote committed Dec 5, 2023
1 parent 317d14a commit 634ae81
Showing 1 changed file with 176 additions and 154 deletions.
Loading

0 comments on commit 634ae81

Please sign in to comment.