https://github.com/GeorgeKA updated 
https://github.com/llvm/llvm-project/pull/118360

>From 3856d6ccf8b10f85cee8d03a349eb768600bd0e0 Mon Sep 17 00:00:00 2001
From: GeorgeKA <k._asa...@hotmail.com>
Date: Mon, 2 Dec 2024 13:34:22 -0500
Subject: [PATCH 1/2] [clang] Document the return value of __builtin_COLUMN

PR for issue #78657

Updated clang/docs/LanguageExtensions.rst to detail the return value of
__builtin_COLUMN for this implementation.
---
 clang/docs/LanguageExtensions.rst | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/clang/docs/LanguageExtensions.rst 
b/clang/docs/LanguageExtensions.rst
index 3c9078bcdf8118..00466fd2e7f020 100644
--- a/clang/docs/LanguageExtensions.rst
+++ b/clang/docs/LanguageExtensions.rst
@@ -4514,9 +4514,13 @@ default member initializer, the invocation point is the 
location of the
 constructor or aggregate initialization used to create the object. Otherwise
 the invocation point is the same as the location of the builtin.
 
-When the invocation point of ``__builtin_FUNCTION`` is not a function scope the
+When the invocation point of ``__builtin_FUNCTION`` is not a function scope, 
the
 empty string is returned.
 
+The builtin ``__builtin_COLUMN`` returns the offset from the start of the line,
+beginning from column 1. `This may differ from other implementations.
+<https://en.cppreference.com/w/cpp/utility/source_location/column>`_
+
 The builtin ``__builtin_source_location`` returns a pointer to constant static
 data of type ``std::source_location::__impl``. This type must have already been
 defined, and must contain exactly four fields: ``const char *_M_file_name``,

>From 2f132e121fb5e4d306bcfe89301e121bdf3b9544 Mon Sep 17 00:00:00 2001
From: GeorgeKA <k._asa...@hotmail.com>
Date: Tue, 10 Dec 2024 11:26:41 -0500
Subject: [PATCH 2/2] Updated the documentation link to the standard instead of
 cppreference

---
 clang/docs/LanguageExtensions.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/clang/docs/LanguageExtensions.rst 
b/clang/docs/LanguageExtensions.rst
index 00466fd2e7f020..33e7f983063b1a 100644
--- a/clang/docs/LanguageExtensions.rst
+++ b/clang/docs/LanguageExtensions.rst
@@ -4519,7 +4519,7 @@ empty string is returned.
 
 The builtin ``__builtin_COLUMN`` returns the offset from the start of the line,
 beginning from column 1. `This may differ from other implementations.
-<https://en.cppreference.com/w/cpp/utility/source_location/column>`_
+<https://eel.is/c++draft/support.srcloc#tab:support.srcloc.current-row-3-column-2-sentence-2>`_
 
 The builtin ``__builtin_source_location`` returns a pointer to constant static
 data of type ``std::source_location::__impl``. This type must have already been

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to