[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-08-09 Thread Timm Bäder via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8065b1cc133c: [clang][Interp] Basic support for bit fields (authored by tbaeder). Changed prior to commit: https://reviews.llvm.org/D155270?vs=540

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-08-01 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D155270#4550473 , @tbaeder wrote: > In D155270#4550466 , @aaron.ballman > wrote: > >> Preco

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-08-01 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. In D155270#4550466 , @aaron.ballman wrote: > Precommit CI is currently crashing on the newly introduced test case. It's missing https://reviews.llvm.org/D155548 :) That's of course just a workaround. At the beginning of the `ev

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-08-01 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Precommit CI is currently crashing on the newly introduced test case. Comment at: clang/test/AST/Interp/bitfields.cpp:36 + // TODO: +=, -=, etc. operators. +} shafik wrote: > This is an interesting test case: > > ``` > struct A

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-08-01 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. Ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155270/new/ https://reviews.llvm.org/D155270 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-07-25 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. Ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155270/new/ https://reviews.llvm.org/D155270 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-07-17 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik added inline comments. Comment at: clang/test/AST/Interp/bitfields.cpp:36 + // TODO: +=, -=, etc. operators. +} This is an interesting test case: ``` struct A {int c:3;}; constexpr int f() { A a1{3}; return a1.c++; } void g() { constexpr int x

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-07-17 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder updated this revision to Diff 540927. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155270/new/ https://reviews.llvm.org/D155270 Files: clang/lib/AST/Interp/ByteCodeExprGen.cpp clang/lib/AST/Interp/ByteCodeStmtGen.cpp clang/lib/AST/Interp/Interp.h clang/lib/AST/Interp/Rec

[PATCH] D155270: [clang][Interp] Basic support for bit fields

2023-07-14 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder created this revision. tbaeder added reviewers: aaron.ballman, erichkeane, shafik, cor3ntin. Herald added a project: All. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews