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

Sat Mar 28 05:55:37 PDT 2020

lebedev.ri added a comment.

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

> In D74156#1924514 <https://reviews.llvm.org/D74156#1924514>, @gchatelet wrote:
>
> > 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.
>
>
> Awesome! :)
>
> > 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?
>
> If that is the preferred direction then i will try to take a look :)
>  Thank you for your patience.

Posted D76921 <https://reviews.llvm.org/D76921>, thanks.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74156