[PATCH] D147890: Fix sysvhash function
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 12 00:46:39 PDT 2023
jhenderson added a comment.
Please remember to upload your patches with full context.
I'll leave @MaskRay to do the review of the logic, based on his discussion on the gABI list.
================
Comment at: llvm/unittests/Object/ELFTest.cpp:275
+
+// Test the sysV and the gnu hash functions, particularly with utf8 unicode
+TEST(ELFTest, Hash) {
----------------
MaskRay wrote:
> utf8 => UTF-8
Comments should end in a trailing "."
================
Comment at: llvm/unittests/Object/ELFTest.cpp:277-278
+TEST(ELFTest, Hash) {
+ // Symbols long enough to have feedback, we have to explicitly encode the utf8
+ // to avoid encoding transliterations.
+ EXPECT_EQ(hashSysV("FooBarBazToto"), 0x5ec3e8fU);
----------------
I'm not sure what this comment is trying to say. What do you mean by "feedback"? Could you reword it a bit, please?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147890/new/
https://reviews.llvm.org/D147890
More information about the llvm-commits
mailing list