[PATCH] D110281: Change __builtin_sycl_unique_stable_name to just use an Itanium mangling

Alexey Bader via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 23 05:08:47 PDT 2021


bader added a comment.

LGTM, just one typo in addition to linter reports and I'd like John to take a look.



================
Comment at: clang/docs/LanguageExtensions.rst:2524
 mangling scheme at runtime. The mangler marks all the lambdas required to name
-the SYCL kernel and emits a stable local ordering of the respective lambdas,
-starting from ``10000``. The initial value of ``10000`` serves as an obvious
-differentiator from ordinary lambda mangling numbers but does not serve any
-other purpose and may change in the future. The resulting pattern is
-demanglable. When non-lambda types are passed to the builtin, the mangler emits
-their usual pattern without any special treatment.
+the SYCL kernel a nd emits a stable local ordering of the respective lambdas.
+The resulting pattern is demanglable.  When non-lambda types are passed to the
----------------
"a nd" -> "and"


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

https://reviews.llvm.org/D110281



More information about the cfe-commits mailing list