[DynInst_API:] [dyninst/dyninst] e4447b: Don't include compiler-specific headers


Date: Wed, 09 Aug 2023 09:21:24 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] e4447b: Don't include compiler-specific headers
  Branch: refs/heads/thaines/remove_cxx_bits_include
  Home:   https://github.com/dyninst/dyninst
  Commit: e4447b379de9c960551dd8a41a537914d6385aa9
      https://github.com/dyninst/dyninst/commit/e4447b379de9c960551dd8a41a537914d6385aa9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-08-09 (Wed, 09 Aug 2023)

  Changed paths:
    M symtabAPI/src/dwarfWalker.h

  Log Message:
  -----------
  Don't include compiler-specific headers

Including anything in <bits/*> is not portable. Also, the contents of
this header were never used.

Stan Cox (scox@xxxxxxxxxx) reported:

It's not included as the first thing in the file, which means it can't
use the precompiled version of that header
(https://gcc.gnu.org/onlinedocs/gcc/Precompiled-Headers.html) and
therefore it's just including the entire C++ standard library, which is
going to be much slower than just including the right headers.


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] e4447b: Don't include compiler-specific headers, Tim Haines <=