[PATCH] D94928: [llvm-mca] Add support for in-order CPUs

[Andrew Savonichev via Phabricator via llvm-commits]

asavonic added inline comments.


================
Comment at: llvm/include/llvm/MC/MCSchedule.h:120
   bool     EndGroup : 1;
+  bool     RetireOOO : 1;
   uint16_t WriteProcResIdx; // First index into WriteProcResTable.
----------------
RKSimon wrote:
> This is a separate problem, but windows builds currently takes 4 bytes for this (before and after this patch) as bool is treated as an int. Would it be possible to change this to:
> ```
>   uint16_t NumMicroOps : 13;
>   uint16_t BeginGroup : 1;
>   uint16_t EndGroup : 1;
>   uint16_t RetireOOO : 1;
> ```
> Ideally replace the bools in an initial patch before this one?
Thanks. Uploaded a fix to https://reviews.llvm.org/D95954.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94928