[PATCH] D101027: [CSKY 7/n] Add more basic instructions including some privilege instructions

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 06:52:32 PDT 2021


rengolin added a comment.

The old 7 is now 9 and there's no 8? This is confusing.

Given that we have already merged the initial part, I think you can stop numbering the patches unless they're meant as a set, which then you start from one again (ex: 1/3, 2/3, 3/3).

Normally we would have gone all the way down small programs being generated (including ALU), but we started wrong, so let's not make it more confusing. :)

However, it's still important to know which patch depends on which, so make sure to still mark the parent/child dependencies correctly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101027



More information about the llvm-commits mailing list