[PATCH] D149068: [llvm-mca][RISCV] Fix checking if data valid in createInstrument

[Andrea Di Biagio via Phabricator via llvm-commits]

andreadb added a comment.

In D149068#4292725 <https://reviews.llvm.org/D149068#4292725>, @michaelmaitland wrote:

>> Though I'm a little surprised it doesn't print a warning or even error for invalid instruments. Is there any reason behind this?
>
> Debug warning is printed on line 85 of `RISCVCustomBehaviour.cpp`. We print an actual error and exit from MCA here <https://github.com/llvm/llvm-project/blob/53791896de3fcc5606c190fa4e4552383ee1dcb8/llvm/tools/llvm-mca/CodeRegionGenerator.cpp#L190>

It would be better if all this custom behaviour logic was properly tested. If I remember it correctly, the plan was to contribute a generic scheduling model as a follow-up to test all of this. Is that still a plan?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149068