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

ClangLoader: Pull out common remapped file operations #3566

Conversation

davemarchevsky
Copy link
Collaborator

I'm making some larger modifications to the loader. While reading
through the do_compile code I noticed that the common "remapped file"
operations - telling various CompilerInvocations about 'virtual'
includes and the virtual main c file - could be factored out to enhance
clarity.

This patch doesn't change functionality at all, nor does it try to make
any opinionated refactoring changes.

I'm making some larger modifications to the loader. While reading
through the `do_compile` code I noticed that the common "remapped file"
operations - telling various CompilerInvocations about 'virtual'
includes and the virtual main c file - could be factored out to enhance
clarity.

This patch doesn't change functionality at all, nor does it try to make
any opinionated refactoring changes.
@davemarchevsky davemarchevsky force-pushed the davemarchevsky_remapped_refactor branch from 2e4bb09 to c8d65dc Compare August 6, 2021 05:48
@yonghong-song
Copy link
Collaborator

[buildbot, test this please]

@davemarchevsky davemarchevsky merged commit b768015 into iovisor:master Aug 6, 2021
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

Successfully merging this pull request may close these issues.

2 participants