[PATCH] D137125: PPC: Implement null target streamer

ChenZheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 18:16:06 PST 2022


shchenz added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCTargetStreamer.h:31
+  virtual void emitAbiVersion(int AbiVersion){};
+  virtual void emitLocalEntry(MCSymbolELF *S, const MCExpr *LocalOffset){};
 };
----------------
Instead of making the base `PPCTargetStreamer` as a null streamer, does it make more sense to add a new child class `PPCTargetNullStreamer`  in `lib/Target/PowerPC/MCTargetDesc/PPCMCTargetDesc.cpp`?


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

https://reviews.llvm.org/D137125



More information about the llvm-commits mailing list