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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 05:34:04 PDT 2023


fhahn added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150310



More information about the llvm-commits mailing list