Patch to Bugzilla 31373

Chad Rosier via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 9 07:45:39 PDT 2017


Hi Erik,

You might consider posting your patch to Phabricator.  You'll likely get 
more feedback/traction there..

https://llvm.org/docs/Phabricator.html

  Chad


On 10/6/2017 8:15 AM, Erik Viktorsson via cfe-commits wrote:
>
> Committing a patch to Bugzilla 31373 
> <https://bugs.llvm.org/show_bug.cgi?id=31373>
>
> A novice programmer so hopefully it complies with the coding policy.
>
> I had to disable an assert in lib/CodeGen/CGExpr.cpp since it requires 
> that all expressions are marked as Used or referenced, which is not 
> possible if we want the ShouldDiagnoseUnusedDecl to return true (thus 
> trigger the warn_unused_variable  warning).
>
> The reason I removed the assert statement is because it causes five 
> tests to fail. These test are the following:
>
> ·clang -cc1 -triple i386-unknown-unknown -mllvm -inline-threshold=1024 
> -O3 -emit-llvm temp-order.cpp
>
> ·clang -cc1 -debug-info-kind=limited -std=c++11 -emit-llvm 
> debug-info-scope.cpp
>
> ·clang -cc1 -std=c++1z -triple x86_64-apple-macosx10.7.0 -emit-llvm 
> cxx1z-init-statement.cpp
>
> ·clang -cc1 -triple x86_64-apple-darwin10 -emit-llvm condition.cpp
>
> ·clang -cc1 -emit-llvm cxx-condition.cpp
>
> /E
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171009/f302334e/attachment.html>


More information about the cfe-commits mailing list