This removes TODO added in http://reviews.llvm.org/D3658
The patch transforms
ABS(NABS(X)) -> ABS(X) NABS(ABS(X)) -> NABS(X)
Differential D4040
Added transform for ABS(NABS(X)) and NABS(ABS(X)) dinesh.d on Jun 6 2014, 12:53 AM. Authored by
Details This removes TODO added in http://reviews.llvm.org/D3658 The patch transforms ABS(NABS(X)) -> ABS(X) NABS(ABS(X)) -> NABS(X)
Diff Detail
Event TimelineComment Actions LGTM I wouldn't bother re-testing all the permutations of the 'icmp' representation -- that code should already be well tested by existing checks? Still, not a big deal, just trying to minimize the redundancy in the testing. Comment Actions Thanks, I had submitted similar patch [http://reviews.llvm.org/D3658] with all permutations of icmp. I am submitting it as it is. But if you care, I can remove few test cases for both changes. |