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

[Javed Absar via Phabricator via llvm-commits]

javed.absar added a comment.

Thanks for this check. 
If ResourceCycles is not specified then it should default to 1 . I believe most people write with that interpretation.
If ResourceCycles is specified for at least one resource, it makes sense from correctness POV that it be specified for all (even though it might get verbose at time).


Repository:
  rL LLVM

https://reviews.llvm.org/D46356