[PATCH] D46821: Update llvm-exegesis to cover latency through another opcode.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 06:59:38 PDT 2018


gchatelet planned changes to this revision.
gchatelet marked 3 inline comments as done.
gchatelet added inline comments.


================
Comment at: tools/llvm-exegesis/lib/BenchmarkRunner.cpp:29
+llvm::BitVector getReservedRegs(const LLVMState &State) {
+  const AssemblerFunction AF(State.createTargetMachine());
+  return AF.ReservedRegs;
----------------
courbet wrote:
> It does not really make sense to create a function to get the reserved registers. Could you refactor this ?
It's part of the redesign of Assembler.{h/cpp}. I'll add a FIXME.


Repository:
  rL LLVM

https://reviews.llvm.org/D46821





More information about the llvm-commits mailing list