On Fri, Oct 6, 2017 at 3:15 PM, Erik Viktorsson via cfe-commits <cfe-commits@lists.llvm.org> wrote: Hi.
> Committing a patch to Bugzilla 31373 > > A novice programmer so hopefully it complies with the coding policy. I think it may be better to put it to Phabricator, see https://llvm.org/docs/Phabricator.html#id3 Please do note that you should generate the diff with context (-U999999), and should put the maillist (cfe-dev in this case) as Subscriber. > > 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 Roman. > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
