[PATCH] D89206: [MemCpyOpt] Add test scaffolding for MSSA based MemCpyOpt

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 10:38:16 PDT 2020


mtrofin added a comment.

In D89206#2358375 <https://reviews.llvm.org/D89206#2358375>, @nikic wrote:

> @mtrofin They show cases where the MSSA and MD implementations produce different results. See the test diffs in D89207 <https://reviews.llvm.org/D89207> for examples.

(The context is D90281 <https://reviews.llvm.org/D90281>)

Is this work in progress, meaning there are more patches coming adding coverage? So for example, in Transforms/MemCpyOpt/vscale-memset.ll, there's no NO_MSSA check right now, but there will be later, is that correct?

If D90281 <https://reviews.llvm.org/D90281> applies, could you comment there about your scenario (i.e. had --check-prefixes been strict, how would this work have been impacted)? Thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89206/new/

https://reviews.llvm.org/D89206



More information about the llvm-commits mailing list