cyndyishida wrote: Talked a bit offline, but the patch LGTM, assuming it is purely restoring the previous behavior before https://github.com/llvm/llvm-project/commit/55323ca6c8b9d21d85591f3499b299b62463321f, which would result in performance parity with the release compiler.
> I suggest we carry this patch downstream (and possibly only in the current > release branch while working to fix the build system or projects in the next > release). I don't have a strong opinion on whether it's merged here or in the fork. But I do believe all changes should go to the development branch too, to avoid the human cost of investigating the same issue as if it's an independently new one. When there's a resolution to the underlying cause, reverts should go to both places as well. https://github.com/llvm/llvm-project/pull/134698 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
