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

gollvm: building fails due to missing header llvm/ADT/None.h #62122

Open
programmingkidx opened this issue Aug 17, 2023 · 3 comments
Open

gollvm: building fails due to missing header llvm/ADT/None.h #62122

programmingkidx opened this issue Aug 17, 2023 · 3 comments
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@programmingkidx
Copy link

What version of Go are you using (go version)?

Not applicable.

Does this issue reproduce with the latest release?

Yes.

What operating system and processor architecture are you using (go env)?

  • Operating system: Ubuntu Linux (kernel 5.15.0.54)
  • Architecture: ARM64
  • Compiler: clang 14.0.0
  • cmake v3.22.1
  • ninja v1.10.1

What did you do?

I followed the directions from this page on building gollvm: https://go.googlesource.com/gollvm
I was at the step to run this command: ninja gollvm

What did you expect to see?

I expected all the files to be compiled successfully.

What did you see instead?

I saw this error message:
llvm-project/llvm/tools/gollvm/passes/GoNilChecks.cpp:28:10: fatal error: 'llvm/ADT/None.h' file not found
#include "llvm/ADT/None.h"
^~~~~~~~~~~~~~~~~
1 error generated.

Additional info:
None.h is located here on my system: /usr/include/llvm-14/llvm/ADT/None.h

@programmingkidx
Copy link
Author

The commit of gollvm I am using is 0f6ef9bc2a14be0a5fdb0d05fc6bb2d5474bf0c6.

@dmitshur dmitshur added this to the gollvm milestone Aug 17, 2023
@dmitshur dmitshur added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Aug 17, 2023
@dmitshur
Copy link
Contributor

CC @ianlancetaylor, @thanm.

@Lantern-r
Copy link

I think it's because the latest llvm-project doesn't include None.h.

llvm/llvm-project@397f2e9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

3 participants