[PATCH] D70999: Fix the macro fusion table for X86 according to Intel optimization manual

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 08:37:06 PST 2019


courbet added inline comments.


================
Comment at: llvm/lib/Target/X86/X86MacroFusion.cpp:52
-  case X86::TEST64ri32:
-  case X86::TEST8mr:
-  case X86::TEST16mr:
----------------
craig.topper wrote:
> courbet wrote:
> > The `mr` versions looked incorrect indeed.
> TEST and CMP mr are loads and are supported. Only the AND/ADD/SUB mr were incorrect
Yes, wrong line, sorry. I though I was reading the ALU versions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70999





More information about the llvm-commits mailing list