[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