[PATCH][DAGCombiner] Factor out new function distributeTruncateThroughAnd

Eric Christopher echristo at gmail.com
Mon Mar 3 14:23:41 PST 2014


In the current incarnation it looks like you could hoist the check for
AND and hasOneUse up out of the function? (Or at least the AND since
it's part of the function name).

-eric

On Mon, Mar 3, 2014 at 2:17 PM, Adam Nemet <anemet at apple.com> wrote:
> Currently this code is duplicated across visitSHL, visitSRA and visitSRL.  The
> plan is to also add rotates as clients to this new function.
>
> There is no functional change intended here.
>
> Adam
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



More information about the llvm-commits mailing list