[PATCH] D62897: [DAGCombine] Initialize the bytes offset vector as INT64_MAX to avoid inference the endian check
Qing Shan Zhang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 19:06:24 PDT 2019
steven.zhang added a comment.
We cannot have the assertion, as it is possible that, it isn't set to something else(for example, this case). And the isBigEndian() check will fail if it isn't set to something else. However, in fact, we could do a check for each element, early return if it isn't set.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62897/new/
https://reviews.llvm.org/D62897
More information about the llvm-commits
mailing list