RE: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Voss, Matthew via cfe-commits
Thanks! LGTM > -Original Message- > From: jkor...@apple.com > Sent: Friday, November 8, 2019 3:33 PM > To: Voss, Matthew > Cc: Jan Korous ; cfe-commits@lists.llvm.org; > jeremy.morse.l...@gmail.com > Subject: Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist

Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Jan Korous via cfe-commits
narrowed it down to your commit on my local machine. Let me know if it >> doesn't repro on your end, though. >>> >>> Thanks, >>> Matthew >>> >>> >>>> -Original Message- >>>> From: jkor...@apple.com >>>>

RE: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Voss, Matthew via cfe-commits
9 2:01 PM > To: Voss, Matthew > Cc: Jan Korous ; cfe-commits@lists.llvm.org; > jeremy.morse.l...@gmail.com > Subject: Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a > dependency in cc1 > > Hi Matthew, > > You were absolutely right - my bad! > > I re

Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Jan Korous via cfe-commits
t; To: Voss, Matthew >> Cc: Jan Korous ; cfe-commits@lists.llvm.org; >> jeremy.morse.l...@gmail.com >> Subject: Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a >> dependency in cc1 >> >> Hi Matthew, >> >> Are you sure it is this commit? It is de

Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Jeremy Morse via cfe-commits
Hi Jan, The x86_64 sanitizer was failing: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24326 And then immediately after the reverts landed: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24327 check-dfsan passes. I don't have any explanation as to wh

RE: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Voss, Matthew via cfe-commits
Voss, Matthew > Cc: Jan Korous ; cfe-commits@lists.llvm.org; > jeremy.morse.l...@gmail.com > Subject: Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a > dependency in cc1 > > Hi Matthew, > > Are you sure it is this commit? It is definitely possible yet sound

Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Jan Korous via cfe-commits
Hi Matthew, Are you sure it is this commit? It is definitely possible yet sounds a bit unlikely that my patch would cause linker errors. Anyway, I am going to reproduce on a linux box. Thanks. Jan > On Nov 7, 2019, at 4:50 PM, Voss, Matthew wrote: > > Hi Jan, > > It looks like this commit

Re: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-08 Thread Jeremy Morse via cfe-commits
Hi Jan, As the dfsan tests have been failing for a bit, I've reverted this in 6b45e1bc, and the follow-up patch in d6be9273c, to clear the buildbots. -- Thanks, Jeremy ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bi

RE: [clang] 03b84e4 - [clang] Report sanitizer blacklist as a dependency in cc1

2019-11-07 Thread Voss, Matthew via cfe-commits
Hi Jan, It looks like this commit is causing DFSAN failures on the sanitizer bots and our internal CI. Could you take a look? http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24312/steps/64-bit%20check-dfsan/logs/stdio Thanks, Matthew > -Original Message- > From: cfe-com