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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to