[PATCH] D32509: Replace HashString algorithm with xxHash64

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 10:01:40 PDT 2017


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

This is fine with me.



================
Comment at: test/DebugInfo/X86/gnu-public-names.ll:218
+; CHECK-NEXT:  [[ANON_INNER]] EXTERNAL TYPE "(anonymous namespace)::inner"
+; CHECK-NEXT:  EXTERNAL FUNCTION "f3"
 ; GCC Doesn't put local statics in pubnames, but it seems not unreasonable and
----------------
aprantl wrote:
> vsk wrote:
> > I'm not familiar with this test, but it's a bit concerning that more checks get added than removed. @aprantl, what are your thoughts on going forward with the test updates vs. making the output of llvm-dwarfdump not depend on the string hash function?
> DWARFDebugPubTable::dump() iterates over the elements of an llvm::Set. We should have separate change first that sorts the set.
Agreed.


Repository:
  rL LLVM

https://reviews.llvm.org/D32509





More information about the llvm-commits mailing list