[PATCH] D34971: Remove freed InvalidDomains from InvalidDomainMap.

Tobias Grosser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 3 21:09:53 PDT 2017


grosser added a comment.

Very nice.

@maxf: Any chance we can get a test case here.

This might already be covered by existing test cases, but is possibly not visible in normal test runs, but only with -DPOLLY_TEST_USE_VG=ON. Can you try this and in this case point out the test cases that already cover this. In case no test case cover this, is it easy to add a test case that covers this in POLLY_TEST_USE_VG mode at least?

If this is the case, we might consider adding a corresponding buildbot.


https://reviews.llvm.org/D34971





More information about the llvm-commits mailing list