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.

Sign up for GitHub

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

Jump to bottom

add-compiler-defaults problems #921

Closed
3 tasks
dkrupp opened this issue Sep 14, 2017 · 0 comments
Closed
3 tasks

add-compiler-defaults problems #921

dkrupp opened this issue Sep 14, 2017 · 0 comments
Assignees
@Xazax-hun
Labels
analyzer 📈 Related to the analyze commands (analysis driver) bug 🐛
Milestone
6.0 pre4

Comments

@dkrupp
Copy link
Member

dkrupp commented Sep 14, 2017

  • --add-compiler-defaults option should cut out the "sysroot" arg from the original build action and call the cross compiler detection in (log_parser.py)

  • auto-detected include paths should be added by -isystem to clang (instead of -I) to get at the end of the inclusion list

  • clang tidy does not get the auto-detected target and include pahts, however it should
@dkrupp dkrupp added this to the 6.0 pre4 milestone Sep 14, 2017
@whisperity whisperity added bug 🐛 analyzer 📈 Related to the analyze commands (analysis driver) labels Sep 14, 2017
@Xazax-hun Xazax-hun mentioned this issue Sep 14, 2017
Merged
dkrupp added a commit that referenced this issue Sep 14, 2017
@dkrupp
Merge pull request #922 from Xazax-hun/CompDetectionImprovements
8768f26 
Compiler detection improvements. Fixes #921
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Xazax-hun Xazax-hun

Labels
analyzer 📈 Related to the analyze commands (analysis driver) bug 🐛
Projects
None yet
Milestone
6.0 pre4
Development

No branches or pull requests
3 participants
@Xazax-hun @whisperity @dkrupp