[PATCH] D48713: [llvm-exegesis] Add uop computation for more X87 instruction classes.
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 5 05:46:32 PDT 2018
courbet added a comment.
Thanks, PTAL.
================
Comment at: tools/llvm-exegesis/lib/X86/Target.cpp:171
static std::vector<llvm::MCInst>
setVectorRegToConstant(const unsigned Reg, const unsigned RegSizeBytes,
const unsigned RMOpcode) {
----------------
gchatelet wrote:
> Reg and RegSizeBytes could theoretically be out of sync. Should RegSizeBytes be computed from Reg?
We could take that RegClass and do RegClass.MC.RegsSize, but is feels awkward.
Repository:
rL LLVM
https://reviews.llvm.org/D48713
More information about the llvm-commits
mailing list