[PATCH] Check for all known bits on ret in InstCombine
Chandler Carruth
chandlerc at google.com
Tue Jul 22 12:56:39 PDT 2014
On Tue, Jul 22, 2014 at 12:48 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> > I'm defining dead as an SSA subgraph where the only sinks (uses which
> > have "side-effects" or otherwise can't be removed) are the
> > assumptions. I think this is sufficiently conservative to remove
> > assumptions from true dead code regions and prevent them from
> > keeping control flow alive without sacrificing optimization
> > opportunities.
>
> But we should already kill them in unreachable blocks.
>
Probably. It also should handle the case where we if-convert prior to
discovering the block was dead, and thus have a dead SSA subgraph without
dead control flow.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140722/b905fbf1/attachment.html>
More information about the llvm-commits
mailing list