[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