[libcxx-commits] [PATCH] D88868: [M680x0] Add CycleTimer implementation for M68K architecture

Thu Dec 10 01:51:47 PST 2020

RKSimon added a comment.

In D88868#2443692 <https://reviews.llvm.org/D88868#2443692>, @myhsu wrote:

> In D88868#2442287 <https://reviews.llvm.org/D88868#2442287>, @lebedev.ri wrote:
>
>> I don't believe this needs libc++'s signoff, it only mechanically replicates
>> the exact same change to the same external dependency into all the duplicate copies of said dependency.
>> @myhsu i still maintain that you can just commit this now.
>
> As @ldionne mentioned libcxx was added automatically and I couldn't remove it. Otherwise I've already committed this patch a while ago: https://github.com/llvm/llvm-test-suite/commit/25883edc2f615a84c4fa741e997904293474bbc0

Isn't that just testsuite's copy of the code from D89011 <https://reviews.llvm.org/D89011>? It looks like at least the libcxx parts of this patch still need committing.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88868/new/

https://reviews.llvm.org/D88868