Skip to content

Conversation

@dmah42
Copy link
Member

@dmah42 dmah42 commented Jun 3, 2021

  • Instead of pulling from the old repo, use the new repo
  • Don't use clang-3.8, use modern clang
  • Update to ubuntu-latest

@google-cla google-cla bot added the cla: yes label Jun 3, 2021
@dmah42
Copy link
Member Author

dmah42 commented Jun 3, 2021

ubuntu-latest + latest libc++ + latest libc++abi + latest clang == failing regex checks on build configuration. ugh.

@dmah42 dmah42 marked this pull request as ready for review June 4, 2021 09:55
@dmah42 dmah42 requested a review from LebedevRI June 4, 2021 09:55
@LebedevRI
Copy link
Collaborator

Seems reasonable

@dmah42 dmah42 merged commit 3424091 into main Jun 4, 2021
@dmah42 dmah42 deleted the modern_san branch June 4, 2021 10:06
dmah42 pushed a commit that referenced this pull request Jun 4, 2021
* Use modern clang/libc++ for sanitizers

* update ubuntu

* new llvm builds differently

* clang, not clang-3.8

* just build what we need
vincenzopalazzo pushed a commit to vincenzopalazzo/benchmark that referenced this pull request Feb 8, 2022
* Use modern clang/libc++ for sanitizers

* update ubuntu

* new llvm builds differently

* clang, not clang-3.8

* just build what we need
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants