Skip to content
This repository was archived by the owner on Jun 5, 2019. It is now read-only.

Build with LLVM 3.5 TOT, whitespace changes in a dedicated commit #3

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

abigagli
Copy link

Hi Stephen, I got a cleaner series of commits now, with all whitespace changes in a dedicated commit.

Review on Reviewable

@CaptainZippy
Copy link

Thanks Andrea! I had a look at the changes again since llvm3.5 was released.

@abigagli
Copy link
Author

No problem,
have a look at the new pull request, and then I can armonize the leading white space if you want.

@stephenatwork
Copy link
Owner

Great, nearly there! Fix the whitespace and we're done.

@abigagli
Copy link
Author

Ok, this should do it.
I still can't understand why, even if passing the proper "-I=" option, it seems it cannot resolve any #include<...> I put in the files being analyzed... I don't have time right now, but if you have any idea/pointer from the top of your head, I'll try to dig further as soon as I can.

@marvin2k
Copy link

same here...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants