Page MenuHomePhabricator

[PATCH 07/27] [noalias] simplify noalias intrinsics
Needs ReviewPublic

Authored by jeroen.dobbelaere on Oct 4 2019, 2:36 PM.

Details

Reviewers
hfinkel
jdoerfert
Summary

This is part of the series started by D68484.

Note: this is a stable point and tests should run fine with the patches applied up to this point.
Note: D68492 has been incorporated.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2019, 2:36 PM

Comments need more work i think. Also, they are phrases, they should be grammatically correct, start with Uppercase letter.

jeroen.dobbelaere marked 2 inline comments as done.Oct 14 2019, 4:54 AM
jeroen.dobbelaere added inline comments.
llvm/lib/Analysis/InstructionSimplify.cpp
5148

As mentioned by a.elivikov in D68492: we should not propagate null through the intrinsics.

5225

Also here, we should not propagate null.

Do not propagate null. Improved tests.

jeroen.dobbelaere retitled this revision from [PATCH 10/38] [noalias] simplify noalias intrinsics to [PATCH 6/26] [noalias] simplify noalias intrinsics.
jeroen.dobbelaere edited the summary of this revision. (Show Details)
jeroen.dobbelaere retitled this revision from [PATCH 6/26] [noalias] simplify noalias intrinsics to [PATCH 06/26] [noalias] simplify noalias intrinsics.Jun 12 2020, 9:18 AM
jeroen.dobbelaere retitled this revision from [PATCH 06/26] [noalias] simplify noalias intrinsics to [PATCH 07/27] [noalias] simplify noalias intrinsics.

Rebased to 9fb46a452d4e5666828c95610ceac8dcd9e4ce16 (September 7, 2020)