[PATCH] D143097: [NFC] add new function is64Bit for SymbolicFile class

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/Object/SymbolicFile.h:161
 
+  virtual bool is64Bit() const { return false; }
+
----------------
jhenderson wrote:
> I think it would be better to make this a pure virtual function and force every implementation to specify the bitness of the object file (even if it's always 32-bit). Otherwise, when a new format comes along, it would be easy to forget to implement this function.
This can be marked done now.


================
Comment at: llvm/include/llvm/Object/TapiFile.h:51-53
+  bool is64Bit() const override { return MachO::is64Bit(Arch); }
 
+  static bool classof(const Binary *v) { return v->isTapiFile(); }
----------------
jhenderson wrote:
> Please leave these methods in the order they were in before.
This can be marked done now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143097