[llvm] [NFC][CodeGen] Improve comments in Target.td (PR #147006)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 4 00:03:19 PDT 2025


https://github.com/jurahul created https://github.com/llvm/llvm-project/pull/147006

- Clarify that InstructionEncoding::Size is size in bytes.
- Fix grammer for `DecoderNamespace` comment.

>From 8a6e5bcf3cdeeca72df69f87aa77660acd2d5856 Mon Sep 17 00:00:00 2001
From: Rahul Joshi <rjoshi at nvidia.com>
Date: Fri, 4 Jul 2025 00:00:49 -0700
Subject: [PATCH] [NFC][CodeGen] Improve comments in Target.td

- Clarify that InstructionEncoding::Size is size in bytes.
- Fix grammer for `DecoderNamespace` comment.
---
 llvm/include/llvm/Target/Target.td | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/llvm/include/llvm/Target/Target.td b/llvm/include/llvm/Target/Target.td
index ce9a2b2751968..4c83f8a580aa0 100644
--- a/llvm/include/llvm/Target/Target.td
+++ b/llvm/include/llvm/Target/Target.td
@@ -538,11 +538,11 @@ include "llvm/Target/TargetInstrPredicate.td"
 include "llvm/Target/TargetSchedule.td"
 
 class InstructionEncoding {
-  // Size of encoded instruction.
+  // Size of encoded instruction in bytes.
   int Size;
 
-  // The "namespace" in which this instruction exists, on targets like ARM
-  // which multiple ISA namespaces exist.
+  // The "namespace" in which this instruction exists (on targets like ARM
+  // where multiple ISA namespaces exist).
   string DecoderNamespace = "";
 
   // List of predicates which will be turned into isel matching code.



More information about the llvm-commits mailing list