Fixed typos

[curdeius]

LGTM. Thanks!

[xgupta]

Thanks @jnyfah for the patch!

It looks you need to rebase your branch to llvm upstream main branch. Because polly/docs/ReleaseNotes.rst is recently change. Now its contain nothing see - https://polly.llvm.org/docs/ReleaseNotes.html.

Release note for the previous branch is now published and we can't change them. So now your patch should contain only the changes in clang/docs/UsersManual.rst and libcxx/docs/UsingLibcxx.rst.

[xgupta]

Oh sorry, Your patch is correct. I misunderstand something :)

[xgupta]

Actually, I am not able to apply this patch with arc patch D100696.

Created and checked out branch arcpatch-D100696.
Checking patch polly/docs/ReleaseNotes.rst...
error: while searching for:

.. warning::

These release notes are for the next release of Polly and it describes
the new features that have recently been committed to our development
branch.

error: patch failed: polly/docs/ReleaseNotes.rst:6
Checking patch libcxx/docs/UsingLibcxx.rst...
Checking patch clang/docs/UsersManual.rst...
Applying patch polly/docs/ReleaseNotes.rst with 1 reject...
Rejected hunk #1.

@curdeius could you help here to understand why this message comes?

[xgupta]

@jnyfah If you are missing something I am just telling you the steps -

1. First fork the llvm github repository
2. git clone https://github.com/jynfah/llvm-project
3. git checkout -b type-fix
4. make changes to files.
5. git diff > mypatch.patch
6. git commit -m "fix typo"
7. go to reviews.llvm.org and login and click on differential then create diff and update the diff (mypatch.patch)
8. complete remaining entries.

Then there is an arc tool way, install it from https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/.
after the 4th step, skip 5 and complete 6, and then do arc diff to create a revision from the terminal. You will be asked to verify credentials.
If reviewers asks you to make changes, checkout your local branch, make changes and then again do arc diff.

Don't forget to rebase by git pull --rebase https://github.com/llvm/llvm-project.git main

[Meinersbur]

polly/docs/ReleaseNotes.rst

9

This diff seems to be based on D100588 which changes "These releaes notes [...] and describe" to "These release notes [...] and it describes". IMHO the second part is incorrect (and reverted by this patch), or at least unnecessary, because 1) "release notes" is plural and 2) "describe" is a compound predicate.

[xgupta]

@jnyfah you may close this revision. changes are committed by https://reviews.llvm.org/rG21bef4e11e48d5d4bff7a23babbd420e86dd420a.

clang/docs/UsersManual.rst

 
 Clang is also capable of generating preprocessed source file(s) and associated
 run script(s) even without a crash. This is specially useful when trying to
 generate a reproducer for warnings or errors while using modules.
 
 .. option:: -gen-reproducer
 
   Generates preprocessed source files, a reproducer script and if relevant, a
-  cache containing: built module pcm's and all headers needed to rebuilt the
+  cache containing: built module pcm's and all headers needed to rebuild the
   same modules.
 
 .. _rpass:
 
 Options to Emit Optimization Reports
 ------------------------------------
 
 Optimization reports trace, at a high-level, all the major decisions

libcxx/docs/UsingLibcxx.rst

 
 
 Using libc++ with GCC
 ---------------------
 
 GCC does not provide a way to switch from libstdc++ to libc++. You must manually
 configure the compile and link commands.
 
-In particular you must tell GCC to remove the libstdc++ include directories
+In particular, you must tell GCC to remove the libstdc++ include directories
 using ``-nostdinc++`` and to not link libstdc++.so using ``-nodefaultlibs``.
 
-Note that ``-nodefaultlibs`` removes all of the standard system libraries and
+Note that ``-nodefaultlibs`` removes all the standard system libraries and
 not just libstdc++ so they must be manually linked. For example:
 
 .. code-block:: bash
 
   $ g++ -nostdinc++ -I<libcxx-install-prefix>/include/c++/v1 \
          test.cpp -nodefaultlibs -lc++ -lc++abi -lm -lc -lgcc_s -lgcc
 
 

 or disable extended libc++ behavior, including enabling "debug mode" or
 thread safety annotations.
 
 **_LIBCPP_DEBUG**:
   See :ref:`using-debug-mode` for more information.
 
 **_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS**:
   This macro is used to enable -Wthread-safety annotations on libc++'s
-  ``std::mutex`` and ``std::lock_guard``. By default these annotations are
+  ``std::mutex`` and ``std::lock_guard``. By default, these annotations are
   disabled and must be manually enabled by the user.
 
 **_LIBCPP_DISABLE_VISIBILITY_ANNOTATIONS**:
   This macro is used to disable all visibility annotations inside libc++.
   Defining this macro and then building libc++ with hidden visibility gives a
   build of libc++ which does not export any symbols, which can be useful when
   building statically for inclusion into another library.
 

polly/docs/ReleaseNotes.rst

 =============================
 Release Notes 13.0 (upcoming)
 =============================
 
 In Polly 13 the following important changes have been incorporated.
 
 .. warning::
 
-  These release notes are for the next release of Polly and it describes
+  These release notes are for the next release of Polly and describe
   the new features that have recently been committed to our development
   branch.
 
 - Change ...