This revision was automatically updated to reflect the committed changes.
Closed by commit rGe0746a8a8d64: [clang] cleanup -fstrict-flex-arrays
implementation (authored by serge-sans-paille).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132944/new/
jyknight added a comment.
(please ignore the last comment, I sent it to the wrong review thread)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132944/new/
https://reviews.llvm.org/D132944
___
cfe-commits mailing list
jyknight added a comment.
#include
#include
consteval int fn(std::source_location sl = std::source_location::current()) {
return sl.line();
}
consteval int fn2(int line = fn()) {
return line;
}
int main() {
printf("fn=%d fn2=%d\n", fn(), fn2());
}
I
serge-sans-paille updated this revision to Diff 456971.
serge-sans-paille edited the summary of this revision.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132944/new/
https://reviews.llvm.org/D132944
Files:
clang/include/clang/Driver/Options.td
clang/lib/CodeGen/CGExpr.cpp
jyknight added inline comments.
Comment at: clang/lib/CodeGen/CGExpr.cpp:910
if (FD->getParent()->isUnion())
-return StrictFlexArraysLevel < 2;
+return true;
RecordDecl::field_iterator FI(
This is a functional change (which is good,
serge-sans-paille created this revision.
serge-sans-paille added a reviewer: jyknight.
Herald added a project: All.
serge-sans-paille requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This is a follow up to https://reviews.llvm.org/D126864,
