Revert "[NFCI][ValueTracking] getUnderlyingObject(): gracefully handle cycles"
authorNikita Popov <nikita.ppv@gmail.com>
Mon, 15 Mar 2021 12:12:39 +0000 (13:12 +0100)
committerNikita Popov <nikita.ppv@gmail.com>
Mon, 15 Mar 2021 12:12:39 +0000 (13:12 +0100)
commit5fb43477dcacd6fca573699dd2691a17eae7dc7e
tree2db41457751f609dbcd055d010d1b83163758c3b
parent9ba577eca2e339726bfaad4e615c6324a705b292
Revert "[NFCI][ValueTracking] getUnderlyingObject(): gracefully handle cycles"

This reverts commit aa440ba24dc25e4c95f6dcf8ff647024f3b12661.

This has a non-trivial compile-time impact:
https://llvm-compile-time-tracker.com/compare.php?from=0c5b789c7342ee8384507c3242fc256e23248c4d&to=aa440ba24dc25e4c95f6dcf8ff647024f3b12661&stat=instructions

I don't believe this is the correct way to address the issue in
this case.
llvm/lib/Analysis/ValueTracking.cpp