Re: r300122 - [clang-cl] Make all sanitizer flags available in clang-cl

> On Apr 13, 2017, at 9:50 AM, Reid Kleckner wrote: > > Fixed in r300209. Was the -Wslash-u-filename warning not explanatory enough > to suggest the solution? Maybe we could add a note like "escape source paths > starting with /U by passing them at the end of the command line after —". Ah, I

Re: r300122 - [clang-cl] Make all sanitizer flags available in clang-cl

Fixed in r300209. Was the -Wslash-u-filename warning not explanatory enough to suggest the solution? Maybe we could add a note like "escape source paths starting with /U by passing them at the end of the command line after --". /Users/buildslave/jenkins/sharedspace/phase1@2/llvm/tools/clang/test/D

Re: r300122 - [clang-cl] Make all sanitizer flags available in clang-cl

I’m afraid I have to revert this to turn the bots green again. > On Apr 12, 2017, at 10:41 PM, Akira Hatanaka via cfe-commits > wrote: > > I think this broke some of the bots. > > http://lab.llvm.org:8080/green/job/clang-stage1-configure-RA_check/30152/ >

Re: r300122 - [clang-cl] Make all sanitizer flags available in clang-cl

I think this broke some of the bots. http://lab.llvm.org:8080/green/job/clang-stage1-configure-RA_check/30152/ Could you take a look? > On Apr 12, 2017, at 3:50 PM, Reid Kleckner via cfe-commits > wrote: > > Author:

r300122 - [clang-cl] Make all sanitizer flags available in clang-cl

Author: rnk Date: Wed Apr 12 17:50:51 2017 New Revision: 300122 URL: http://llvm.org/viewvc/llvm-project?rev=300122&view=rev Log: [clang-cl] Make all sanitizer flags available in clang-cl Summary: Use a tablegen let {} block so that new sanitizer flags are available by default in all driver modes