Well, if they worked I wasn't going to say we needed to add tests for them, i just wanted to make sure they work before we move onto something else. In any case, lgtm
On Fri, Sep 8, 2017 at 4:43 PM Bob Haarman via Phabricator < revi...@reviews.llvm.org> wrote: > inglorion updated this revision to Diff 114463. > inglorion added a comment. > > added examples suggested by @zturner, verified step over and step into > specific behavior matches MSVC, and added tests for them > > > https://reviews.llvm.org/D37529 > > Files: > clang/lib/CodeGen/CGDebugInfo.cpp > clang/lib/CodeGen/CodeGenModule.h > clang/test/CodeGenCXX/debug-info-nested-exprs.cpp > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits