[PATCH] D78129: Add Marvell ThunderX3T110 support
Wei Zhao via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 6 13:00:41 PDT 2020
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
More information about the cfe-commits
mailing list