[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