[PATCH] Enhance boolean simplification to handle more cases

[Michael Liao]

committed in r179265 with those lines of code removed.

- michael

On Wed, 2013-04-10 at 20:05 -0500, Arnold Schwaighofer wrote:
> On Apr 10, 2013, at 7:03 PM, Michael Liao <michael.liao at intel.com> wrote:
> 
> > On Wed, 2013-04-10 at 18:59 -0500, Arnold Schwaighofer wrote:
> >> Incidentally, I have been working on a very similar patch today. You beat me to it. :P
> >> 
> >> I tried the test cases from your patch and my patch does not catch all of them (I am guessing because I am not flexible on which side the constant to the AND is). So let me review your patch.
> >> 
> >> +static SDValue checkBoolTestSetCCCombine(SelectionDAG &DAG,
> >> 
> >> I don't think the extra parameter is really needed. You only use it for debug output as far as I can tell.
> >> 
> >> +  DEBUG(dbgs() << "Unhandled node in boolean test checking: ");
> >> +  DEBUG(SetCC.getNode()->dump(&DAG));
> >> +  DEBUG(dbgs() << "\n");
> >> 
> >> This looks like debugging output from while you were developing, can you remove it?
> > 
> > I intentionally leave it there and want to capture some cases may need
> > handling in the future. I am not sure whether it's OK. If not, I will
> > leave them in my private branch.
> 
> You would be the only one looking for it and the only one who would understand why it is there. I think it is better kept in your private repo.
> 
> - Arnold