[PATCH] D76096: [clang] allow const structs to be constant expressions for C

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 21 13:38:41 PDT 2023


efriedma added a comment.

My primary concern here is making sure we don't actually blow up compile-time.   D151587 <https://reviews.llvm.org/D151587> fixes the dependency from CGExprConstant, which was the most complicated one, but there are other places that call into Evaluate().  Some of those are probably unavoidable, but I'd like to make sure at least that we don't end up evaluating the initializers of global variables in C code.

Not sure there's any good way to regression-test that, though...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76096



More information about the cfe-commits mailing list