Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D64386

CodeGen: Use memset in initializers for non-zeros
AbandonedPublic
Actions

Authored by vitalybuka on Jul 8 2019, 10:08 PM.

Details

Reviewers
eugenis
pcc
jfb
efriedma
hfinkel

Diff Detail

Event Timeline

vitalybuka created this revision.Jul 8 2019, 10:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 8 2019, 10:08 PM
Herald added subscribers: cfe-commits, dexonsmith. · View Herald Transcript
Harbormaster completed remote builds in B34564: Diff 208596.Jul 8 2019, 10:08 PM
vitalybuka added a parent revision: D64382: Use getMostFrequentByte to decide if should used memset+stores.Jul 8 2019, 10:08 PM
vitalybuka added a child revision: D64387: CodeGen: Do not memset if following stores will overwrite entire variable.Jul 8 2019, 10:57 PM
xbolva00 added reviewers: efriedma, hfinkel.Jul 24 2019, 7:50 AM
hfinkel added inline comments.Jul 24 2019, 9:00 AM
clang/test/CodeGen/init-memset.c
29

I'd be more comfortable if the test here actually checked the length and the value in the memset call here and below.

vitalybuka abandoned this revision.Jun 16 2021, 3:18 PM

Revision Contents
Changeset List

PathSize
clang/
lib/
CodeGen/
5 lines
test/
CodeGen/
5 lines

Diff 208596

View Options

clang/lib/CodeGen/CGDecl.cpp

Loading...
View Options

clang/test/CodeGen/init-memset.c

Loading...