Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix failure in buildbot #86661

Merged
merged 1 commit into from
Mar 26, 2024
Merged

Fix failure in buildbot #86661

merged 1 commit into from
Mar 26, 2024

Conversation

zahiraam
Copy link
Contributor

@zahiraam zahiraam commented Mar 26, 2024

@zahiraam zahiraam changed the title Fix failure in buildbot, builds/25788. Fix failure in buildbot Mar 26, 2024
@zahiraam zahiraam marked this pull request as ready for review March 26, 2024 16:19
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Mar 26, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Mar 26, 2024

@llvm/pr-subscribers-clang

Author: Zahira Ammarguellat (zahiraam)

Changes

Sanitizer failure in https://lab.llvm.org/buildbot/#/builders/19/builds/25788


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

1 Files Affected:

  • (modified) clang/lib/AST/TypePrinter.cpp (+4-4)
diff --git a/clang/lib/AST/TypePrinter.cpp b/clang/lib/AST/TypePrinter.cpp
index d9504f9dcb3899..d0ba6066fa0180 100644
--- a/clang/lib/AST/TypePrinter.cpp
+++ b/clang/lib/AST/TypePrinter.cpp
@@ -2303,10 +2303,10 @@ printTo(raw_ostream &OS, ArrayRef<TA> Args, const PrintingPolicy &Policy,
     } else {
       if (!FirstArg)
         OS << Comma;
-        // Tries to print the argument with location info if exists.
-        printArgument(Arg, Policy, ArgOS,
-                      TemplateParameterList::shouldIncludeTypeForArgument(
-                          Policy, TPL, ParmIndex));
+      // Tries to print the argument with location info if exists.
+      printArgument(Arg, Policy, ArgOS,
+                    TemplateParameterList::shouldIncludeTypeForArgument(
+                        Policy, TPL, ParmIndex));
     }
     StringRef ArgString = ArgOS.str();
 

@zahiraam zahiraam mentioned this pull request Mar 26, 2024
@zahiraam zahiraam merged commit b069131 into llvm:main Mar 26, 2024
9 checks passed
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.

None yet

2 participants