aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp
diff options
context:
space:
mode:
authorAiden Grossman <agrossman154@yahoo.com>2023-08-17 13:57:51 -0700
committerAiden Grossman <agrossman154@yahoo.com>2023-08-18 13:15:12 -0700
commit28134a29fdedd8972acdfb39223571ddcc15dc59 (patch)
tree50a109b6f3c6cce4ee6875520193878a5b304653 /clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp
parent4da76ea70ad3d225d495703efca9b2f56bdefbf6 (diff)
downloadllvm-28134a29fdedd8972acdfb39223571ddcc15dc59.zip
llvm-28134a29fdedd8972acdfb39223571ddcc15dc59.tar.gz
llvm-28134a29fdedd8972acdfb39223571ddcc15dc59.tar.bz2
[NFCi][MergeFunctions] Consolidate Hashing Functions
A couple years ago, StructuralHash was created, copying the exact hashing implementation from FunctionComparator (minus a couple small details/refactorings). Since then, the hashing implementation has not diverged, but several other areas, like unit testing, have diverged significantly, with StructuralHash getting more attention in these areas. This patch aims to consolidate the two hashing functions into StructuralHash given they do the exact same thing and having less divergence in areas like unit testing would be beneficial. The original aim at creating a separate StructuralHash was to make the implementation divergent and capture additional details like instruction operands (which neither hashing implementation does currently). The MergeFunctions pass doesn't need these detaisl, but verification of pass return values would benefit from this additional data. Setting an option to calculate these values would allow for divergent behavior where appropriate while reducing code duplication with little runtime overhead. Reviewed By: aeubanks Differential Revision: https://reviews.llvm.org/D158217
Diffstat (limited to 'clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp')
0 files changed, 0 insertions, 0 deletions