Skip to content

[IR] Fix IWYU violation #93918

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
May 31, 2024
Merged

Conversation

cyndyishida
Copy link
Member

@cyndyishida cyndyishida commented May 31, 2024

GEPNoWrapFlags.h calls assert creating a undeclared identifier error when running an Apple-stage2 build with LLVM_ENABLE_MODULES enabled.

resolves: rdar://129031201

GEPNoWrapFlags.h calls `assert` creating a undeclared identifier error
when running an Apple-stage2 build with LLVM_ENABLE_MODULES enabled.
@llvmbot
Copy link
Member

llvmbot commented May 31, 2024

@llvm/pr-subscribers-llvm-ir

Author: Cyndy Ishida (cyndyishida)

Changes

GEPNoWrapFlags.h calls assert creating a undeclared identifier error when running an Apple-stage2 build with LLVM_ENABLE_MODULES enabled.


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

1 Files Affected:

  • (modified) llvm/include/llvm/IR/GEPNoWrapFlags.h (+2)
diff --git a/llvm/include/llvm/IR/GEPNoWrapFlags.h b/llvm/include/llvm/IR/GEPNoWrapFlags.h
index feaccc878de0c..4d456cc691032 100644
--- a/llvm/include/llvm/IR/GEPNoWrapFlags.h
+++ b/llvm/include/llvm/IR/GEPNoWrapFlags.h
@@ -13,6 +13,8 @@
 #ifndef LLVM_IR_GEPNOWRAPFLAGS_H
 #define LLVM_IR_GEPNOWRAPFLAGS_H
 
+#include <assert.h>
+
 namespace llvm {
 
 /// Represents flags for the getelementptr instruction/expression.

@francisvm
Copy link
Collaborator

Thank you!

@cyndyishida cyndyishida merged commit 4985f25 into llvm:main May 31, 2024
9 checks passed
@cyndyishida cyndyishida deleted the eng/PR-moduleFixes branch May 31, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants