[clang] [llvm] Enable fexec-charset option (PR #138895)

Sergei Barannikov via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 10 17:14:22 PST 2026


================
@@ -507,6 +507,9 @@ class Triple {
   /// For example, "fooos1.2.3" would return "1.2.3".
   LLVM_ABI StringRef getEnvironmentVersionString() const;
 
+  /// getDefaultNarrowTextEncoding - Get the default encoding of the triple.
----------------
s-barannikov wrote:

Quote from the coding standard:
> Don’t duplicate the function or class name at the beginning of the comment

Also, the comment could be more verbose, "default encoding of the triple" is confusing without a context.

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


More information about the cfe-commits mailing list