[PATCH] D112206: [CSKY] First patch to construct codegen infra and generate first add instruction

[Renato Golin via Phabricator via llvm-commits]

rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

Thanks again, this looks good to me now. Please wait a day or so to merge, in case there are any more reviews.



================
Comment at: llvm/lib/Target/CSKY/MCTargetDesc/CSKYBaseInfo.h:26
+// instruction info tracks. All definitions must match CSKYInstrFormats.td.
+namespace CSKYII {
+
----------------
zixuan-wu wrote:
> rengolin wrote:
> > What's the relation of this with `CSKYInstrInfo`?
> > 
> > How do other targets share that kind of stuff?
> Sorry to can't understand what you mean. I think this comments are similar meaning as RISCVBaseInfo.h has.
I see, I was comparing this to the Arm/x86 back-ends. Last time I worked on this level was too long ago, and there is precedent, so this should be fine. :)


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

https://reviews.llvm.org/D112206