[PATCH] D150310: [TableGen][SubtargetEmitter] Add the StartAtCycles field in the WriteRes class.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli added a comment.

In D150310#4341950 <https://reviews.llvm.org/D150310#4341950>, @fhahn wrote:

> This patch is causing multiple warnings when building with assertions.
>
>   [3147/3173] Building CXX object unittests/tools/llvm-exegesis/CMakeFiles/LLVMExegesisTests.dir/X86/SchedClassResolutionTest.cpp.o
>   ...//llvm-project/llvm/unittests/tools/llvm-exegesis/X86/SchedClassResolutionTest.cpp:68:70: warning: missing field 'StartAtCycle' initializer [-Wmissing-field-initializers]
>         computeIdealizedProcResPressure(STI.getSchedModel(), {{P0Idx, 2}});
>
> It would be great if you could take a look. If it takes longer to fix, please revert the  commit in the meantime.

Thank you for the heads up @fhahn  - I have created a fix at https://reviews.llvm.org/D150569

Francesco


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150310