check

remove white space

fix test and clang format

rebase, test passing locally, in build stage 1

In D125944#3689022, @v.g.vassilev wrote:

@Purva-Chaudhari can you rebase this patch?

Rebase

Rebase

In D125944#3689022, @v.g.vassilev wrote:

@Purva-Chaudhari can you rebase this patch?

modify tests

add preamble to test file

Updating

File name

Added new file for template test

Asan tests passing! Solves the memory leak issue for original as well as modified version of execute.cpp (Commit issue solved )

In D127991#3589949, @sunho wrote:

@Purva-Chaudhari Could you check this out? Basically, this should allow even long execute.cpp test case to be ran without asan failures.

In D127945#3588719, @sunho wrote:

Hi @Purva-Chaudhari I haven't set up asan build setting yet. It's sad to hear it's still mem leaking. We need to look into somewhere else then...

Hi @sunho Did you test the changes after stage 2 asan build? The execute.cpp fails in my asan build area after adding this patch

Fix asan test fail

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 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 the commit before passed:
https://lab.llvm.org/buildbot/#/builders/5/builds/24220

Thanks a lot for the notice @uabelho! I have reverted the patch while we are investigating.

@Purva-Chaudhari, can you take look and reproduce the asan failure locally?

Shifted private member