https://github.com/eugeneepshteyn approved this pull request.
https://github.com/llvm/llvm-project/pull/137752
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
eugeneepshteyn wrote:
I have no opinion. If you need it there and it works, go a head and merge...
https://github.com/llvm/llvm-project/pull/137752
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
https://github.com/eugeneepshteyn approved this pull request.
https://github.com/llvm/llvm-project/pull/133223
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
eugeneepshteyn wrote:
> Can it be more made smaller to specifically target the issue at hand?
No, these changes must go in together. In any case, the old PR was submitted a
few weeks ago and at least has advantage of being used. I wouldn't have the
same level of confidence for any new PR and w