search

ponylang / ponyc

Pony is an open-source, actor-model, capabilities-secure, high performance programming language
http://www.ponylang.io
BSD 2-Clause "Simplified" License
5.73k stars 415 forks source link

Allow multiple sanitizers to be enabled at the same time #4538

Closed dipinhora closed 22 hours ago

dipinhora commented 1 day ago

Allow multiple sanitizers to be enabled at the same time

SeanTAllen commented 1 day ago

Is there anything we can do as a simple test to make sure this doesn't get broken in the future?

dipinhora commented 1 day ago

we can add a CI build matrix variation that enables multiple sanitizers (asan/ubsan probably).. it's something to work towards where we can have both enabled as a CI matrix option with them configured to fail if anything is flagged as a way to maintain C code hygiene.. unfortunately, it's not really feasible as of right now due to a large amount of preexisting issues that would cause all such builds to fail...

not sure if it is possible to add in a test for CMake logic otherwise..