https://github.com/ldionne commented:
I just saw https://github.com/llvm/llvm-project/pull/151651 -- I probably should have looked at that one before this PR, but they appeared in the wrong order in my list. I do agree with what @philnik777 said in that other review -- I think we need additional context and a RFC to understand what's your underlying plan, what the constraints are and evaluate what is the impact to libc++. Don't get me wrong, we care about security related proposals so we're likely to want to do something to facilitate what you're doing. But I think we need a high level discussion before we start reviewing individual patches that we (the maintainers) don't fully understand the end destination of. https://github.com/llvm/llvm-project/pull/151652 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
