[llvm-bugs] [Bug 50527] New: llvm/ADT/DenseMapInfo.h isn't layered right

via llvm-bugs llvm-bugs at lists.llvm.org
Sat May 29 09:49:04 PDT 2021


https://bugs.llvm.org/show_bug.cgi?id=50527

            Bug ID: 50527
           Summary: llvm/ADT/DenseMapInfo.h isn't layered right
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Support Libraries
          Assignee: unassignedbugs at nondot.org
          Reporter: clattner at nondot.org
                CC: llvm-bugs at lists.llvm.org

DenseMapInfo.h is an uber header for defining the densemap hashing templates,
including pointers and strings and is included by a ton of stuff.  It is pulled
in by DenseMap.h as well as tons of other stuff.

The problem is that it pulls in both APSInt.h and APInt.h in order to implement
their DenseMapInfo specializations.  It would be much better to move these
specializations to APInt.h and APSInt.h to avoid having almost everything in
LLVM pull in infrequently used types like APSInt.h.

-Chris

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20210529/1633b388/attachment.html>


More information about the llvm-bugs mailing list