balazske added a comment.

In D77809#1999050 <https://reviews.llvm.org/D77809#1999050>, @xazax.hun wrote:

> Actually, sorry. I just realized that the alignof problem is introduced by 
> this patch. I'd love to see the solution committed together with this patch 
> (it could be a separate patch but preferably, they should be committed 
> together.)


I do not see that the alignof problem is introduced here. If this patch is 
ignored the alignof problem still exists. I have found the problematic code, 
for every kind of `UnaryExprOrTypeTraitExpr` the VLA size is evaluated if there 
is a VLA. It looks easy to fix, just add a few lines and tests for it. Still it 
is a not related problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77809



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

Reply via email to