[llvm] [NFC] Annotate CommentFlag with underlying type (PR #186560)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 15 09:54:00 PDT 2026


================
@@ -393,7 +393,7 @@ class MachineInstr
   void clearAsmPrinterFlags() { AsmPrinterFlags = 0; }
 
   /// Return whether an AsmPrinter flag is set.
-  bool getAsmPrinterFlag(CommentFlag Flag) const {
+  bool getAsmPrinterFlag(uint8_t Flag) const {
----------------
boomanaiden154 wrote:

Is there a reason not to prefer `CommentFlag` here and in the `clearAsmPrinterFlag` declaration below to improve readability?

https://github.com/llvm/llvm-project/pull/186560


More information about the llvm-commits mailing list