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

Wed May 2 08:06:18 PDT 2018

javed.absar added a comment.

In https://reviews.llvm.org/D46356#1085333, @courbet wrote:

> In https://reviews.llvm.org/D46356#1085328, @RKSimon wrote:
>
> > I think it'd be clearer if we could keep Btver2/Znver1 to the Res = [1] default tbh
>
>
> Hm, not sure how I can do that without triggering the check for  cases when the user erroneously specified `[1]` when they wanted to use the default (now `[]`). Did I miss a possibility ?
>  This makes is no difference for the user, but allows more checking. Best of both worlds ? :)

I am bit confused on - "user erroneously specified `[1]` when they wanted to use the default ". But the default was always [1]

Repository:
  rL LLVM

https://reviews.llvm.org/D46356