search

PCRE2Project / pcre2

PCRE2 development is now based here.
Other
917 stars 191 forks source link

Test clang with and without optimisations #559

Closed NWilson closed 1 week ago

NWilson commented 1 week ago

The build failure here is due to #558

zherczeg commented 1 week ago

Please update this patch to master. After it is landed, the next step for me is #553

NWilson commented 1 week ago

Grr, there's still a build problem, even after your fix. I'll try and work it out as quick as possible.

It may be that running clang -fsanitize -O0 simply isn't sensible.

carenas commented 1 week ago

It may be that running clang -fsanitize -O0 simply isn't sensible

yes, whichs is why I suggested doing it on windows instead (without. the sanitize part, of course)

NWilson commented 1 week ago

@zherczeg I've fixed it; the next build should pass, and will be OK to merge.

zherczeg commented 1 week ago

Ok. Lets wait the CI then.