[PATCH] D64385: GodeGen, NFC: Add test to track emitStoresForConstant behavior

Vitaly Buka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 10 15:35:51 PDT 2019


vitalybuka marked an inline comment as done.
vitalybuka added inline comments.


================
Comment at: clang/test/CodeGen/init-memset.c:36
+  int a[16] = {0, 0, 1};
+  // CHECK: call void @llvm.memset.{{.*}}
+  // CHECK: store i32 1
----------------
eugenis wrote:
> Would it be useful to check the range of the memset? I.e. does it overlap with the following store or not.
I don't change ranges to it will only complicate tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64385





More information about the cfe-commits mailing list