This is an archive of the discontinued LLVM Phabricator instance.

[AssumeBundles] Preserve information in EarlyCSE
ClosedPublic

Authored by Tyker on Mar 25 2020, 6:15 AM.

Details

Summary

this patch preserve information from various places in EarlyCSE into assume bundles.

Diff Detail

Event Timeline

Tyker created this revision.Mar 25 2020, 6:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 25 2020, 6:15 AM
jdoerfert added inline comments.Mar 25 2020, 8:28 AM
llvm/include/llvm/IR/KnowledgeRetention.h
37

Now that I see this I wonder why we have captial letters in the beginning.

llvm/test/Transforms/EarlyCSE/guards.ll
4

Can you please add the check lines and run the update script first in a local NFC commit. diff against that one and commit it just prior to this patch once accepted?

Tyker updated this revision to Diff 252674.Mar 25 2020, 2:31 PM
Tyker marked an inline comment as done.

Addressed comments.

This revision is now accepted and ready to land.Mar 25 2020, 10:13 PM
This revision was automatically updated to reflect the committed changes.