Page MenuHomePhabricator

[PATCH 11/38] [noalias] D9377: llvm.noalias - don't block EarlyCSE
AbandonedPublic

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

Details

Reviewers
hfinkel
jdoerfert
Summary

This is part of the series started by D68484.

Rebase of D9377

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2019, 2:37 PM
lebedev.ri added inline comments.
llvm/lib/Transforms/Scalar/EarlyCSE.cpp
937

This looks incorrect to me.
Did you mean Intrinsic::noalias ?
Or does Intrinsic::noalias identify as Intrinsic::assume?

jeroen.dobbelaere marked an inline comment as done.Oct 6 2019, 12:16 PM
jeroen.dobbelaere added inline comments.
llvm/lib/Transforms/Scalar/EarlyCSE.cpp
937

Yes, that should have been Intrinsic::noalias.

Fix bad rebase.

For review convenience, all changes related to the original local restrict patches have now been incorporated in the full restrict patch series.