Skip to content

Comments

Re-enable asan testing#231

Closed
steven-johnson wants to merge 7 commits intomasterfrom
srj/enable-asan
Closed

Re-enable asan testing#231
steven-johnson wants to merge 7 commits intomasterfrom
srj/enable-asan

Conversation

@steven-johnson
Copy link
Contributor

No description provided.

'LLVM_ENABLE_RTTI': 'ON',
'LLVM_ENABLE_TERMINFO': 'OFF',
'LLVM_ENABLE_WARNINGS': 'ON', # silence them, it's not like we're gonna fix them
'LLVM_ENABLE_WARNINGS': 'OFF', # silence them, it's not like we're gonna fix them
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love the old misleading comment

@alexreinking
Copy link
Member

This PR is stale. Funny enough, ASAN testing was re-enabled. And then I recently had to re-disable it. I am working on re-enabling it now. What a circus 🙂

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