rsandifo-arm added inline comments.
================ Comment at: clang/include/clang/Basic/AttributeCommonInfo.h:55 + + /// The attibute has no source code manifestation and is only created + /// implicitly. ---------------- erichkeane wrote: > If I recall, there was some pretty awful funny business in some attributes, > which would explicitly use '0' instead of AS_GNU as implicit. Did you run > into any of these? > > Would it make sense to make AS_Implicit 'first' here to catch those? Or > perhaps make '0' ill-formed (and assert?) and make this '1'? Thanks for the reviews! Bumping the values to 1 sounds good to me. I've created https://reviews.llvm.org/D148148 for that. I kept AS_GNU first due to: ``` // Note TableGen depends on the order above. Do not add or change the order // without adding related code to TableGen/ClangAttrEmitter.cpp. ``` (I don't know whether that still applies, but it seemed better to keep the tablegen-sensitive stuff at “one end” of the enum.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148101/new/ https://reviews.llvm.org/D148101 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
