ASan blacklist
    Johan Engelen 
    j at j.nl
       
    Sun Dec 31 13:10:50 UTC 2017
    
    
  
On Sunday, 31 December 2017 at 12:20:32 UTC, kinke wrote:
> On Wednesday, 27 December 2017 at 19:32:25 UTC, Johan Engelen 
> wrote:
>> Hi all,
>>   I want to add ASan blacklists to our repo as Jon Degenhardt 
>> suggested, so we can collaborate on working towards 
>> ASan-enabled standard libraries.
>> What would you prefer?
>> Option 1:  create separate blacklists for phobos and druntime, 
>> and put them in the submodules
>> Option 2: one blacklist for all, put it in LDC's runtime 
>> directory.
>> Option 3: something else.
>
> No strong opinion about this; I think it also depends on the 
> number of blacklist entries. I'd expect way more entries for 
> druntime; in case there'd be none for Phobos at all, I'd put 
> them in the druntime repo.
After thinking a bit more about it, I feel the blacklist should 
go into the repo that contains the functions it is blacklisting. 
That way, the codefix+blacklist-removal  (or code annotation and 
blacklist update from unchecked section to checked) happens in 
one commit, instead of two commits in different repos.
Indeed, I also expect that eventually there shouldn't be any 
blacklist necessary for Phobos code. So then we'll just remove it.
(btw, I think this should be upstreamed at some point; druntime 
and Phobos upstream should have permanent ASan testing upstream 
too)
-Johan
    
    
More information about the digitalmars-d-ldc
mailing list