[PATCH] D48876: [X86][BtVer2][MCA][NFC] Add CMPEQ one-idioms tests

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 4 09:27:50 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D48876#1152421, @andreadb wrote:

> >> You can commit those tests to show that we don't correctly model dependency breaking packed compare instructions on BtVer2. However, I would remove the `padd` from the tests.
> > 
> > @andreadb do you feel like officially LGTM'ing this modulo the nit? :)
>
> Yes please. Thanks!


Thank you!


Repository:
  rL LLVM

https://reviews.llvm.org/D48876





More information about the llvm-commits mailing list