[PATCH] D50398: Update msbuild integration warnings: Don't warn on /Zi and /X
Zachary Turner via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 7 11:04:23 PDT 2018
Lgtm
On Tue, Aug 7, 2018 at 1:42 PM Nico Weber via Phabricator <
reviews at reviews.llvm.org> wrote:
> thakis created this revision.
> thakis added reviewers: zturner, hans.
>
> We do need to map /Zi to /Z7 explicitly for msbuild as explained in this
> file, but since /Zi is passed by default and since things transparently
> work fine with it mapped to /Z7, we shouldn't produce effectively
> inactionable noise for it.
>
> Also don't warn on /X since clang-cl supports that (since r326357; the
> risk of duplicating a bunch of clang-cl driver logic here).
>
>
> https://reviews.llvm.org/D50398
>
> Files:
> llvm/tools/msbuild/LLVM.Cpp.Common.targets
>
>
> Index: llvm/tools/msbuild/LLVM.Cpp.Common.targets
> ===================================================================
> --- llvm/tools/msbuild/LLVM.Cpp.Common.targets
> +++ llvm/tools/msbuild/LLVM.Cpp.Common.targets
> @@ -40,10 +40,7 @@
> to look at the tool output.
> -->
> <Target Name="BeforeClCompile" BeforeTargets="ClCompile">
> - <!-- Warn on /Zi and /ZI, then map them both to /Z7. -->
> - <Warning Condition="'%(ClCompile.DebugInformationFormat)' ==
> 'ProgramDatabase'"
> - File="@(ClCompile)(0,0)"
> - Text="clang-cl does not support /Zi (Program Database). The
> file will be compiled as if /Z7 (C7 Compatible Debug Info) had been
> passed. Update the Debug Information Format in project settings to silence
> this warning."/>
> + <!-- Warn on /ZI, then map to /Z7. /Zi is mapped to /Z7 below too
> (see comment there), but don't warn on that. -->
> <Warning Condition="'%(ClCompile.DebugInformationFormat)' ==
> 'EditAndContinue'"
> File="@(ClCompile)(0,0)"
> Text="clang-cl does not support /ZI (Program Database for Edit
> and Continue). The file will be compiled as if /Z7 (C7 Compatible Debug
> Info) had been passed. Update the Debug Information Format in project
> settings to silence this warning."/>
> @@ -58,11 +55,6 @@
> File="@(ClCompile)(0,0)"
> Text="clang-cl does not support MSVC Link Time
> Optimization. Disable this option in compatibility settings to silence
> this warning."/>
>
> - <!-- Warn if Ignore Standard Include Paths is non-empty, then ignore
> it. -->
> - <Warning Condition="'%(ClCompile.IgnoreStandardIncludePath)' ==
> 'true'"
> - File="@(ClCompile)(0,0)"
> - Text="clang-cl does not support Ignore Standard Include Path
> (/X). Disable this option in compatibility settings to silence this
> warning."/>
> -
> <!-- Warn if Smaller Type Check is enabled, then ignore it.-->
> <Warning Condition="'%(ClCompile.SmallerTypeCheck)' == 'true'"
> File="@(ClCompile)(0,0)"
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180807/7bbe2b03/attachment.html>
More information about the llvm-commits
mailing list