Skip to content

Conversation

@Sirraide
Copy link
Member

Many editors and IDEs automatically delete trailing whitespace on save, and this particular one has shown up as an unrelated change in several of my patches that I then had to remove later (and I’ve seen it in other people’s patches too); this has wasted too much of my time, so I’m removing it separately.

@Sirraide Sirraide merged commit eb40a01 into llvm:main Oct 29, 2025
6 of 10 checks passed
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Oct 29, 2025
@Sirraide Sirraide deleted the fix-trailing-whitespace branch October 29, 2025 01:50
@llvmbot
Copy link
Member

llvmbot commented Oct 29, 2025

@llvm/pr-subscribers-clang

Author: None (Sirraide)

Changes

Many editors and IDEs automatically delete trailing whitespace on save, and this particular one has shown up as an unrelated change in several of my patches that I then had to remove later (and I’ve seen it in other people’s patches too); this has wasted too much of my time, so I’m removing it separately.


Full diff: https://github.com/llvm/llvm-project/pull/165498.diff

1 Files Affected:

  • (modified) clang/include/clang/Parse/Parser.h (+1-1)
diff --git a/clang/include/clang/Parse/Parser.h b/clang/include/clang/Parse/Parser.h
index 0d2316f73fb62..dad8efd0f017f 100644
--- a/clang/include/clang/Parse/Parser.h
+++ b/clang/include/clang/Parse/Parser.h
@@ -7677,7 +7677,7 @@ class Parser : public CodeCompletionHandler {
   /// [GNU] asm-clobbers:
   ///         asm-string-literal
   ///         asm-clobbers ',' asm-string-literal
-  /// \endverbatim 
+  /// \endverbatim
   ///
   StmtResult ParseAsmStatement(bool &msAsm);
 

aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
Many editors and IDEs automatically delete trailing whitespace on save,
and this particular one has shown up as an unrelated change in several
of my patches that I then had to remove later (and I’ve seen it in other
people’s patches too); this has wasted too much of my time, so I’m
removing it separately.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:frontend Language frontend issues, e.g. anything involving "Sema" clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants