tree a9d9ec051663e2f57ae705e5189c2ce4f024bd9c
parent 8e7b25bb401ada4e1d5d026712150ab43598a79e
author Roman Lebedev <lebedev.ri@gmail.com> 1585933187 +0300
committer Roman Lebedev <lebedev.ri@gmail.com> 1585934104 +0300
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCgAdFiEEjkF6151RK40WXe2HCDw+u0oWieAFAl6HbxgACgkQCDw+u0oW
 ieB7BA//dRdvJ9F3xGzJEnrY6CrjhNpV16zN7aDZL0zUjeyHgCtP05aYfiB3TFr9
 fJ6VVoxGHn7dEeW0ZZeka+7tGqeLxKl2MTONJEVD0bmS1k5IBZ/3YLsF5c1DTfpJ
 Z2xG76zJ87NLtN8xJI80beeUK+q6oeslAszTRZPNaGz9KhfMq5vgWX56xGvjgLIM
 pCWQAMMvURFw0vAZObZhFNUfNEqwKIFA1hFkwgSFcyHlJAy5a5rzOcSnWiYTvuFQ
 /Vtdt0Ifq6xqZpNoAPSjky3jYX/Dj7rzA7UZvrY61mU64SUZ3BIgBGmLRMF4Tbet
 XaNy/SDw/wtn4U0S51tUg65EHEhpxMEotQO7ur6JBAiMDEuP3R9LAkXAQ1kesMk6
 FgBB3PJPhlCjZtSyvx8U8T6ByebgwYgrfPAaAAuCiJrdAPq4J4HLB5Mey1eQdaLu
 FAgzZ7Q6XGyMjbVu1K/c4Tl9BHu5O7Dvf+6lP6qA2NHwWvlWmI21TzUubUjQsTqO
 PtZbYLY6+CQxDnjjZtWjo3TjNoABNgtQzzdGRW52StwXUJV277phwtwgIC73Pp2Y
 fh8e4MXQw8sZKPZfYmsMG7vZbul6BYcm8VVz4Vo7Pwg8L7esyu+gALkqtYjAC5Sa
 GeHry2E7gfUpQvKY7PGBGe8IJ8biuiZbw1Zu+q0WcTEMdGT+lAc=
 =iMW7
 -----END PGP SIGNATURE-----

Revert "[SCEV] rewriteLoopExitValues(): even if have hard uses, still rewrite if cheap (PR44668)"

As discussed in post-commit review in https://reviews.llvm.org/D73501
if the goal of this is to help vectorizer, then we should actually
be teaching vectorizer to do this, because right now this rewrite
is still budget-limited, which isn't what we'd want.

Additionally, while the rest of the patch series was universally profitable,
this particular patch is reportedly (https://reviews.llvm.org/D73501#1905171)
exposing cost-modeling issues on ARM.

So let's just back this particular patch out. Once there's an undo transform,
this could be considered for reintegration.

This reverts commit 44edc6fd2c63b7db43e13cc8caf1fee79bebdb5f.