[PATCH] D51751: Merge clang's isRepeatedBytePattern with LLVM's isBytewiseValue

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 20 15:56:16 PDT 2018


jfb added a comment.

In https://reviews.llvm.org/D51751#1237272, @efriedma wrote:

> Missing test coverage for half.
>
> Missing test coverage here for ConstantStruct/ConstantArray/ConstantVector, although I guess the array/struct bits are covered by the clang patch.  I'd like to see some coverage for vectors with unusual element sizes, though, like `<i1 x 16>`.


I added some tests. Indeed the clang-side test/CodeGenCXX/auto-var-init.cpp already exercises a bunch of this.


Repository:
  rL LLVM

https://reviews.llvm.org/D51751





More information about the llvm-commits mailing list