Apply suggestions from code review

Co-authored-by: Louis Dionne <[email protected]>

Author: poor-circle

libcxx/docs/ReleaseNotes/22.rst

@@ -116,7 +116,8 @@ ABI Affecting Changes
   potentially inheriting from the types they wrap. At this point in time we are not aware of any ABI changes caused by
   this.
 
-- ``align`` has been removed from libc++ library in ABI v2. We also add a new ABI flag ``_LIBCPP_ABI_DO_NOT_EXPORT_ALIGN``.
+- ``std::align`` is now implemented as an inline function and its definition is removed from the
+   libc++ built library in ABI v2, or when the ``_LIBCPP_ABI_DO_NOT_EXPORT_ALIGN`` ABI configuration is enabled..
 
 Build System Changes
 --------------------

libcxx/include/__configuration/abi.h

@@ -105,7 +105,7 @@
 #endif
 
 // The PE/COFF format reports a linking error when encountering multiple symbol definitions where at least one is a
-// strong symbol. So we can't inlining a non-inline function without ABI break change.
+// strong symbol. So we can't inline a function that also has a non-inline definition visible.
 #if defined(_LIBCPP_OBJECT_FORMAT_COFF)
 #  define _LIBCPP_DISABLE_INLINE_OPTIMIZE_BECAUSE_MULTIPLY_SYMBOLS_ERROR
 #endif

libcxx/src/memory_align.cpp

@@ -10,7 +10,7 @@
 #include <__cstddef/size_t.h>
 #include <cstdint>
 
-// Don't include <memory> to avoid mulitple declartion of align()
+// Don't include <memory> to avoid multiple declarations of std::align()
 
 #if !defined(_LIBCPP_ABI_DO_NOT_EXPORT_ALIGN)