Daniil Kovalev
Daniil Kovalev
- Addressed the review comments except https://github.com/llvm/llvm-project/pull/72714/#discussion_r1423345089 (I'll submit a subsequent commit addressing that shortly). - Fixed merge conflict appeared after #77300, see a021f15540300e032446825de805143f0f6214c4.
@MaskRay Published updates on all issues you've mentioned - would be glad to see your comments on new changes.
@MaskRay A kind reminder regarding the PR
@MaskRay Would be glad to see your comments on changes addressing your comments
@MaskRay would be glad to see your review on the changes
@MaskRay > Is there a dynamic loader implementation? Without it it's difficult to verify the correctness. Yes, there is: see https://github.com/access-softek/musl/tree/dkovalev/pauth-code-drop. I've used that to verify the correctness, and it...
@MaskRay Thanks for your feedback, see answers to your comments and new commits addressing the issues. I want to bring attention to [discussion_r1495128934](https://github.com/llvm/llvm-project/pull/72714/#discussion_r1495128934) - I've discovered a case with large...
@MaskRay Would be glad to see your feedback on the changes
@MaskRay Thanks for feedback! I've addressed the comments. Regarding > processAux should be simplified Please see my comment in [discussion_r1517190622](https://github.com/llvm/llvm-project/pull/72714#discussion_r1517190622). I might miss something, but I don't feel that `processAux`...
@MaskRay Would be glad to see your feedback on the latest changes