[PATCH] D104734: [llvm-diff] Explicitly check ConstantStructs for differences
John McCall via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 22 12:03:49 PDT 2021
rjmccall added inline comments.
================
Comment at: llvm/tools/llvm-diff/DifferenceEngine.cpp:477
+ if (CSL->getType()->getNumElements() != CSR->getType()->getNumElements())
+ return false;
+
----------------
Hmm. I don't think it's okay to just compare the number of elements: if the types might be different, they might have different layouts.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104734/new/
https://reviews.llvm.org/D104734
More information about the llvm-commits
mailing list