[PATCH] D101921: [MC] Make it possible for targets to define their own MCObjectFileInfo

Philipp Krones via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 7 01:00:38 PDT 2021


flip1995 added a comment.

In D101921#2743735 <https://reviews.llvm.org/D101921#2743735>, @MaskRay wrote:

> Can you post the RISCV patch depending on this one?

Yes, sure. See https://reviews.llvm.org/D102052.



================
Comment at: llvm/include/llvm/Support/TargetRegistry.h:1029
 
+/// RegisterMCObjectFileInfo - Helper template for registering a target object
+/// file info implementation.  This invokes the static "Create" method on the
----------------
MaskRay wrote:
> https://llvm.org/docs/CodingStandards.html#doxygen-use-in-documentation-comments
> "Don’t duplicate function or class name at the beginning of the comment."
> 
> Just ignore existing comments which do not abide by the standard.
> https://llvm.org/docs/CodingStandards.html#doxygen-use-in-documentation-comments
> "Don’t duplicate function or class name at the beginning of the comment."
> 
> Just ignore existing comments which do not abide by the standard.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101921



More information about the cfe-commits mailing list