glider marked 4 inline comments as done.
glider added inline comments.
Herald added a subscriber: jdoerfert.


================
Comment at: tools/clang/test/CodeGenCXX/auto-var-init.cpp:496
 // ZERO-LABEL: @test_smallpartinit_uninit()
 // ZERO: call void @llvm.memset{{.*}}, i8 0,
 
----------------
jfb wrote:
> I wonder if (maybe in a separate patch) you also want to avoid `memset` when 
> something is pretty small.
If I'm understanding your request correctly, the same code also handles the 
memset() case for zero-initialization.


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

https://reviews.llvm.org/D57898



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to