[PATCH] D46356: [TableGen] Emit a fatal error on inconsistencies in resource units vs cycles.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 2 05:25:18 PDT 2018


RKSimon added a comment.

Not sure if I want to have to specify resource cycles every time - we should be able to assume a default=1, same for latency and micro-ops. This helps reduce bulkiness of the models considerably. In fact I might go further and maybe suggest a debug warning that values set to the default value could be removed?

But I'd definitely happy with the idea that if ResourceCycles are set it must be equal in size to ResourceUnits, (and similar ambiguities).


Repository:
  rL LLVM

https://reviews.llvm.org/D46356





More information about the llvm-commits mailing list