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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 20 16:10:49 PDT 2018


efriedma added inline comments.


================
Comment at: test/Transforms/MemCpyOpt/memcpy-to-memset.ll:54
+
+ at i1x16 = internal constant <16 x i1> <i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0, i1 0>, align 4
+define void @test_i1x16() nounwind {
----------------
You're not really testing any interesting codepaths with a constant zero?  We just hit the "isNullValue()" early exit.


Repository:
  rL LLVM

https://reviews.llvm.org/D51751





More information about the llvm-commits mailing list