[PATCH] D60193: [OpenCL] Added addrspace_cast operator

Argyrios Kyrtzidis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 21 16:12:45 PDT 2020


akyrtzi added inline comments.
Herald added a subscriber: dexonsmith.


================
Comment at: clang/include/clang-c/Index.h:2057
+   */
+  CXCursor_CXXAddrspaceCastExpr = 129,
+
----------------
Hi Anastasia, apologies for not catching this earlier, but libclang is intended to keep a stable ABI and changing the enumerator values breaks libclang's ABI compatibility.

Would you be able to make a follow-up commit and restore the enumerator values to their original values? I would suggest to add `CXCursor_CXXAddrspaceCastExpr` at the end and assign to it the next available value that is not already taken.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60193



More information about the cfe-commits mailing list