[PATCH] D121497: Lex: add support for `{,u}i128` Microsoft extension

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 15 10:47:38 PDT 2022


rnk added a comment.

Seems reasonable, this could use far more testing.



================
Comment at: clang/test/Lexer/ms-extensions.c:27
+#define INT128_MAX  170141183460469231731687303715884105727i128
+#define UINT128_MAX 0xffffffffffffffffffffffffffffffffui128
+
----------------
This seems like it could use more testing. Check for other `__int128` tests, and try adding a few of these literals to those tests with some basic arithmetic expressions.

In a similar vein, I believe there are already bugs that lurk behind `__int128` non-type template parameters. It's worth setting up a few of those test cases. One of the main usages of the expr printer is to produce debug info for non-type template parameters. Do i128 suffixes come through there correctly now?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121497



More information about the cfe-commits mailing list