[llvm] [AsmParser][MCA] Fix handling of multi-character comments (PR #147228)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 6 23:03:25 PDT 2025


================
@@ -0,0 +1,10 @@
+	// REQUIRES: aarch64-registered-target
----------------
MaskRay wrote:

This comment still looks vague... `//` is the universal comment marker, but AArch64 sets `CommentString` to `//`, so it will print `//`.
However, it seems that this MC test passes even before this change.

I don't think `// RUN: llvm-mc -preserve-comments -n -triple aarch64 %s -o %t` (removed unneeded `<` and used `-o` for more conventional tests) needs to be indented

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


More information about the llvm-commits mailing list