[PATCH] D124228: [NFC][NewGVN][LoadCoercion] Add new tests for future commit

Konstantina Mitropoulou via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 14 17:29:09 PDT 2022


kmitropoulou added a comment.

In D124228#3577315 <https://reviews.llvm.org/D124228#3577315>, @nlopes wrote:

> There's no tradition in sharing tests between GVN & NewGVN. Are you going to implement the functionality in both new & old GVN?
>
> Plus, we would appreciate if you used poison instead of undef. Thank you!

I replaced all the undef values with poison.

To my understanding the new and old gvn have different lit tests. The old GVN already supports load coercion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124228



More information about the llvm-commits mailing list