[PATCH] D74156: [llvm-exegesis] Exploring X86::OperandType::OPERAND_COND_CODE

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 07:35:35 PDT 2020


gchatelet added a comment.

In D74156#1920632 <https://reviews.llvm.org/D74156#1920632>, @lebedev.ri wrote:

> ... did that explanation of the question i'm having made any sense?


Thx for digging in the conversation !
Ok it makes more sense now.

I discussed it a bit with @courbet:

- We want the analysis tool to stay simple so we'd rather not make it knowledgeable of the repetition mode.
- We'd like to still be able to select either repetition mode to dig into special cases

So we could add a third `min` repetition mode that would run both and take the minimum. It could be the default option.
Would you have some time to look what it would take to add this third mode?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74156





More information about the llvm-commits mailing list