[PATCH] D93372: [CSKY 3/n] Add bare-bones C-SKY MCTargetDesc

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 03:21:02 PST 2020


rengolin added inline comments.


================
Comment at: llvm/lib/Target/CSKY/MCTargetDesc/CSKYAsmBackend.h:12
+
+#include "MCTargetDesc/CSKYMCTargetDesc.h"
+#include "llvm/MC/MCAsmBackend.h"
----------------
zixuan-wu wrote:
> rengolin wrote:
> > Do you really need this include here?
> Yes, class MCTargetOptions is introduced by it.
Why not include `"llvm/MC/MCTargetOptions.h"` directly here?

Your target description header has some auto-generated code that could get really large.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93372



More information about the llvm-commits mailing list