[PATCH] D27129: [SelectionDAG] Add partial BITCAST support to computeKnownBits

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 05:57:08 PST 2016


RKSimon added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:2176
+
+      bool IsLE = getDataLayout().isLittleEndian();
+      for (unsigned i = 0; i != SubScale; ++i) {
----------------
efriedma wrote:
> No testcase for a big-endian target?
I'm struggling to create a big-endian test case for this (its only happening on x86 with use of a particular 'extending' intrinsic). So I'm proposing we don't do this for non little endian targets for now.


Repository:
  rL LLVM

https://reviews.llvm.org/D27129





More information about the llvm-commits mailing list