[libc-commits] [PATCH] D72516: [llvm-libc] Add memory function benchmarks
Guillaume Chatelet via Phabricator via libc-commits
libc-commits at lists.llvm.org
Mon Jan 13 01:20:51 PST 2020
gchatelet added a comment.
In D72516#1814843 <https://reviews.llvm.org/D72516#1814843>, @abrachet wrote:
> This is a very big patch and I don't have too much time today so I haven't reviewed this all and I focused on llvm style for now. A lot of comments I only made once but apply throughout.
>
> I know something was posted on the mailing list today and I'm guessing this was talked about offline before work started here but I wish this was done more incrementally and discussed on the lists before. This is pretty hard to review as it is.
My apologies for this, we wanted to have the patch and the post ready at the same time so people could refer to each other and get a better idea of the proposal.
It's not too late to split this into multiple patches if you feel like it's worth it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72516/new/
https://reviews.llvm.org/D72516
More information about the libc-commits
mailing list