drodriguez wrote:

> @drodriguez What do you think about merging this PR to the release branch?

@nikic I imagine this is you asking and not a sentient robot? :D

I think the fix is safe enough for cherry-picking into the release branch. A 
couple of people in the issue, PR, and related issues I have found had 
confirmed it works for them. Not sure if it classifies as important enough for 
cherry-picking into the release branch, but it is true that creates binaries 
that might be rejected from some tooling (but not the dynamic loaded itself in 
my experience).

In summary: not opposed to cherry-picking into the release branch, but also I 
will not be offended if this is not considered important enough.

https://github.com/llvm/llvm-project/pull/131398
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to