[PATCH] D65148: [SimplifyCFG] Bump phi-node-folding-threshold from 2 to 3

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 15 11:08:04 PDT 2019


dmgreen added a comment.

Fair enough.

One last issue, which might come up from this change. This time with a multiply, that was previously able to simplify one of the compares to true, I think in CVP.  Same __SSAT as before.

  void arm_mult_q7(const signed char * pSrcA, const signed char * pSrcB, signed char * pDst, unsigned  blockSize)
  {
    unsigned  blkCnt = blockSize;
    while (blkCnt > 0U)
    {
      *pDst++ = (signed char) __SSAT((((short) (*pSrcA++) * (*pSrcB++)) >> 7), 8);
      blkCnt--;
    }
  }

i.e this: https://rise4fun.com/Alive/fplbG


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65148/new/

https://reviews.llvm.org/D65148





More information about the llvm-commits mailing list