[PATCH] D153180: [docs][TableGen][Target] Improve the documentation of the attribute value for SubtargetFeature.

Paul Kirth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 16:51:09 PDT 2023


paulkirth added a comment.

Thanks so much for adding this! This is definitely LGTM from me, but I'll let your other reviewers decide if anything else should be added.



================
Comment at: llvm/docs/WritingAnLLVMBackend.rst:1768
 
+If the value for the attribute is the string "true" or "false", the attribute
+is assumed to be a bool and only one SubtargetFeature should refer to it.
----------------
Do you think it would help to reference which parameter you're referring to in the example code?

Also, do you think it bears mentioning how the boolean value is determined? It is certainly an implementation detail, but it may be good to specify.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153180



More information about the llvm-commits mailing list