Tyker marked an inline comment as done. Tyker added inline comments.
================ Comment at: cfe/trunk/lib/Parse/ParseDecl.cpp:3939 + "both or neither of isAlreadyConsumed and " + "RangeEnd needs to be set"); + DS.SetRangeEnd(isAlreadyConsumed ? RangeEnd : Tok.getLocation()); ---------------- RKSimon wrote: > @Tyker @rsmith We're seeing -Wparentheses warnings because of this - please > can you take a look? The logic looks a little dodgy for a both/neither > assertion as well. > > http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/27322/steps/build%20stage%201/logs/warnings%20%281%29 i made a revision that fixes this: https://reviews.llvm.org/D61731 Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60934/new/ https://reviews.llvm.org/D60934 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
