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

llvm on OSX #4

Closed
jonbaer opened this issue May 11, 2017 · 5 comments
Closed

llvm on OSX #4

jonbaer opened this issue May 11, 2017 · 5 comments

Comments

@jonbaer
Copy link

jonbaer commented May 11, 2017

OSX 10.11.6 + Homebrew 1.2 (+ using mapd-deps-osx.sh)

GroupByAndAggregate.h:34:10: fatal error:
'llvm/IR/Function.h' file not found
#include <llvm/IR/Function.h>

Any ideas for a fix? Looking at http://stackoverflow.com/questions/26246769/including-llvm-value-h-not-found do I need a different version of llvm (4.0)?

@asuhan
Copy link
Contributor

asuhan commented May 11, 2017

Do you have llvm-config in your path? Version 4.0 should be fine.

@jonbaer
Copy link
Author

jonbaer commented May 11, 2017

Thanks, I am up + running, seems that brew --prefix got confused on an older copy while I was trying to figure it out (4.0 is fine) ...

brew --prefix llvm
/usr/local/Cellar/llvm/4.0.0_1
(previous)
brew --prefix llvm@3.7
/usr/local/opt/llvm@3.7

@bluealert
Copy link

I have removed llvm using brew remove llvm and the directory "llvm" is also deleted. but when I run brew --prefix llvm, the output is always: /usr/local/Cellar/llvm/5.0.0.

Does someone know the reason? How can I change the output of brew --prefix llvm ?

@jonbaer
Copy link
Author

jonbaer commented Sep 22, 2017

This seems to happen every LLVM release where they push the previous version down into /usr/local/opt for some reason. I think what should happen is the mapd-deps-osx.sh script should use llvm-config --prefix (vs. brew --prefix) - your output for --prefix is stuck at which version is linked.

You can try the following:
brew unlink llvm
brew install llvm
brew link --force llvm

@bluealert
Copy link

@jonbaer Thank you.

I run brew help --prefix, it's output is as follow:

brew --prefix:
    Display Homebrew's install path. *Default:* /usr/local

brew --prefix formula:
    Display the location in the cellar where formula is or would be installed.

alexbaden pushed a commit that referenced this issue Jan 2, 2020
andrewseidl pushed a commit that referenced this issue Nov 9, 2021
* Register local / global hint in Calcite

* Support g_ prefix for global query hint name

* Translate global hint in analyzer

* Add tests

* Apply comments #1: global hint registration

* Apply comments #2: global hint flag identification

* Apply comments #3: global hint translation

* Apply comments #4: remove unnecessary virtual keyword

* Fixup a bug on allow_gpu_hashtable build hint for overlaps join

* Fixup a bug related to a query having multiple identical subqueries

* Add global hint tests related to overlaps join hashtable

* Apply comments #5: misc cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants