[PATCH] D141772: [clang][Interp] Fix record initialization via CallExpr subclasses
    Aaron Ballman via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Jan 18 10:16:58 PST 2023
    
    
  
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
> I think the patch itself is mostly obvious, but I was wondering if it is the right time to add such RUN lines to existing test cases like I do here for test/SemaCXX/constexpr-nqueens.cpp.
If the tests are (mostly) passing without a bunch of FIXME comments, then yes, I think it's a great step to start running more of these tests with both constant expression engines.
LGTM!
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141772/new/
https://reviews.llvm.org/D141772
    
    
More information about the cfe-commits
mailing list