[clang] [llvm] [CodeGen][KCFI] Switch type hash from xxHash64 to FNV-1a (PR #167254)
Sami Tolvanen via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 10 07:33:46 PST 2025
================
@@ -3186,7 +3186,7 @@ void CodeGenModule::finalizeKCFITypes() {
continue;
std::string Asm = (".weak __kcfi_typeid_" + Name + "\n.set __kcfi_typeid_" +
- Name + ", " + Twine(Type->getZExtValue()) + "\n")
+ Name + ", " + Twine(Type->getSExtValue()) + "\n")
----------------
samitolvanen wrote:
I seem to recall that this was intentional, and was needed for the values to be usable in x86 relocations. Did you confirm that kernel builds still work with this change?
https://github.com/llvm/llvm-project/pull/167254
More information about the llvm-commits
mailing list