[PATCH] D93576: [openacc] Use TableGen enum for default clause value

[sameeran joshi via Phabricator via llvm-commits]

sameeranjoshi added inline comments.


================
Comment at: llvm/include/llvm/Frontend/OpenACC/ACC.td:84
+def ACC_Default_none : ClauseVal<"none", 1, 0> { let isDefault = 1; }
+def ACC_Default_present : ClauseVal<"present", 0, 0> {}
+
----------------
Trying to understand - 
`class ClauseVal<string n, int v, bit uv>` mentions second parameter (`i.e v`)as value of clause.

Is there are reason to start the numbering from `0`?
In `OMP.td` as well I see they always start with index `2`, any defined pattern used here?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93576