Skip to content

[MLIR][LSP][NFC] Fix a header guard #84862

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

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

walter-erquinigo
Copy link
Member

This header guard is wrong and conflicts with the one from Transport.h

This header guard is wrong and conflicts with the one from Transport.h
@llvmbot llvmbot added mlir:core MLIR Core Infrastructure mlir labels Mar 12, 2024
@llvmbot
Copy link
Member

llvmbot commented Mar 12, 2024

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-core

Author: Walter Erquinigo (walter-erquinigo)

Changes

This header guard is wrong and conflicts with the one from Transport.h


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

1 Files Affected:

  • (modified) mlir/include/mlir/Tools/lsp-server-support/SourceMgrUtils.h (+2-2)
diff --git a/mlir/include/mlir/Tools/lsp-server-support/SourceMgrUtils.h b/mlir/include/mlir/Tools/lsp-server-support/SourceMgrUtils.h
index 969058b0228895..9ed8326a602e6c 100644
--- a/mlir/include/mlir/Tools/lsp-server-support/SourceMgrUtils.h
+++ b/mlir/include/mlir/Tools/lsp-server-support/SourceMgrUtils.h
@@ -11,8 +11,8 @@
 //
 //===----------------------------------------------------------------------===//
 
-#ifndef MLIR_TOOLS_LSPSERVERSUPPORT_TRANSPORT_H
-#define MLIR_TOOLS_LSPSERVERSUPPORT_TRANSPORT_H
+#ifndef MLIR_TOOLS_LSPSERVERSUPPORT_SOURCEMGRUTILS_H
+#define MLIR_TOOLS_LSPSERVERSUPPORT_SOURCEMGRUTILS_H
 
 #include "mlir/Tools/lsp-server-support/Protocol.h"
 #include "llvm/Support/SourceMgr.h"

@walter-erquinigo walter-erquinigo merged commit e4a5467 into llvm:main Mar 12, 2024
@walter-erquinigo walter-erquinigo deleted the walter/fix branch July 6, 2024 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mlir:core MLIR Core Infrastructure mlir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants