<div dir="ltr"><div style>The attached patch essentially reverts r159230.</div><div style><br></div><div style>The reasons why are because:</div><div style>1. It creates an asymmetry in the IR by hurting our ability to canonicalize things like C - zext(X) and C + zext(!X) into the same thing.</div>
<div style>2. Adding it back does not break any tests and without it we still generate the inferior code mentioned in the commit log.</div><div style><br></div><div style>It seems like we should keep the canonicalization and fix (the apparently still broken problem) in the lowering stage.</div>
<div style>The commit mentioned that some targets don't have a sext i1. This shouldn't be a concern at the IR level, should it?</div><div style><br></div><div style>-- </div><div style>David Majnemer</div></div>