https://github.com/zyn0217 commented:
LGTM in general, but my concern would be that we're less clear if there's any use of this function out-of-tree, so as an alternative, can we turn the implementation to use HeuristicResolver and explicitly deprecate it for a while before eventually removing it? https://github.com/llvm/llvm-project/pull/128392 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
