Page MenuHomePhabricator

Introduce a generic memset to bzero transformation in CodeGen
Needs ReviewPublic

Authored by gchatelet on Oct 31 2019, 9:21 AM.

Details

Reviewers
courbet
Summary

This is WIP and superseding D69623

Event Timeline

gchatelet created this revision.Oct 31 2019, 9:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 31 2019, 9:21 AM
resistor added inline comments.
llvm/lib/Analysis/TargetLibraryInfo.cpp
55

This name reads like it refers to functions that were added in 2001, not to functions that were removed after 2001.

llvm/test/CodeGen/AArch64/arm64-memset-to-bzero.ll
11

Can you remove these unrelated changes?

gchatelet updated this revision to Diff 281056.Jul 27 2020, 2:30 PM

rebase and revert unrelated changes.

gchatelet marked an inline comment as done.Jul 27 2020, 2:30 PM

A bunch of tests are still failing

LLVM :: CodeGen/AArch64/arm64-memset-to-bzero-pgso.ll
LLVM :: CodeGen/AArch64/arm64-memset-to-bzero.ll
LLVM :: CodeGen/Mips/indirect-jump-hazard/long-calls.ll
LLVM :: CodeGen/Mips/micromips-target-external-symbol-reloc.ll
LLVM :: CodeGen/X86/force-align-stack-alloca.ll
LLVM :: CodeGen/X86/pr36865.ll
LLVM :: CodeGen/X86/stack-clash-unknown-call.ll
LLVM :: Transforms/InferFunctionAttrs/annotate.ll