[PATCH] D78129: Add Marvell ThunderX3T110 support

[Wei Zhao via Phabricator via llvm-commits]

wxz2020 added a comment.

I can confirm your findings, basically as long as we comment out the new instructions, it will be OK to build.

Agree with your changes 1), 2) and 3).

4. this seems does not matter to build, as long as we comment out the new instructions in 5), setting it "1" or "0", both are OK. they all can be compiled

5. if we comment out these instructions, then we will not be able to schedule them efficiently in the instruction scheduler. I know in other models or earlier versions, they could be treated as NOP, but in our new machine, they do have a timing effect, 7 cycles to finish. Any way to represent this in .md?

Thanks a lot for the help,


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78129