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

Correctly set resource dir when not specified explicitly #1171

Closed
Xazax-hun opened this issue Nov 23, 2017 · 0 comments
Closed

Correctly set resource dir when not specified explicitly #1171

Xazax-hun opened this issue Nov 23, 2017 · 0 comments
Assignees
Labels
analyzer 📈 Related to the analyze commands (analysis driver) bug 🐛
Milestone

Comments

@Xazax-hun
Copy link
Contributor

Right now we detect default include directories of gcc. This might interfere with clang resource dirs. We need to include the clang resource dir with higher priority than the detected gcc includes (but potentially lower than user include dirs?). To address the priority issue we specify the clang resource dir with the -isystem flag. This, however, is not specified when the resource dir is not set in the package layout. From clang 5.0 it is easy to query the resource dir from clang. We should query the resource dir from clang in case it is not specified in the package layout config.

@Xazax-hun Xazax-hun added this to the release 6.2.1 milestone Nov 23, 2017
@whisperity whisperity added bug 🐛 analyzer 📈 Related to the analyze commands (analysis driver) labels Nov 23, 2017
martong pushed a commit to martong/codechecker that referenced this issue Nov 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer 📈 Related to the analyze commands (analysis driver) bug 🐛
Projects
None yet
Development

No branches or pull requests

3 participants