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

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 4 09:22:44 PDT 2018


andreadb accepted this revision.
andreadb added a comment.
This revision is now accepted and ready to land.

>> 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!


Repository:
  rL LLVM

https://reviews.llvm.org/D48876





More information about the llvm-commits mailing list