[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-06-11 Thread Purva Chaudhari via Phabricator via cfe-commits
Purva-Chaudhari updated this revision to Diff 436129. Purva-Chaudhari added a comment. Fix asan test fail CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123674/new/ https://reviews.llvm.org/D123674 Files: clang/lib/Interpreter/IncrementalParser.cpp clang/test/Interpreter/errorRecove

[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-05-31 Thread Purva Chaudhari via Phabricator via cfe-commits
Purva-Chaudhari added a comment. In D123674#3546738 , @v.g.vassilev wrote: > In D123674#3546722 , @uabelho wrote: > >> Hi, >> >> I noticed that the testcase Interpreter/execute.cpp starts failing with this >> pa

[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-05-30 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev added a comment. In D123674#3546722 , @uabelho wrote: > Hi, > > I noticed that the testcase Interpreter/execute.cpp starts failing with this > patch when run/compiled with asan: > > Failed Tests (1): > Clang :: Interpreter/execute.cpp

[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-05-30 Thread Mikael Holmén via Phabricator via cfe-commits
uabelho added a comment. Hi, I noticed that the testcase Interpreter/execute.cpp starts failing with this patch when run/compiled with asan: Failed Tests (1): Clang :: Interpreter/execute.cpp Seen in the buildbot run https://lab.llvm.org/buildbot/#/builders/5/builds/24221 The run on th

[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-05-28 Thread Vassil Vassilev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG5ff27fe1ff03: [clang-repl] Recover the lookup tables of the primary context. (authored by Purva-Chaudhari, committed by v.g.vassilev). Herald added a project: clang. Changed prior to commit: https://rev

[PATCH] D123674: Clang-Repl Error Recovery Bug Fix

2022-05-28 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev accepted this revision. v.g.vassilev added a comment. This revision is now accepted and ready to land. LGTM! Apologies for the delay. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123674/new/ https://reviews.llvm.org/D123674 ___