dwblaikie wrote:

> Should we add a test for this flag? Something around existing tests in 
> clang/test/CXX/temp/temp.spec/temp.expl.spec/p2-20.cpp and 
> clang/test/CXX/dcl.dcl/dcl.spec/dcl.stc/p1.cpp.

Oh, right, I did have a new test I meant to add, but that's a better place for 
it - so I've added that test coverage in 
https://github.com/llvm/llvm-project/pull/96699/commits/b3facc9dffc762e3cef93886df23026d502afcae

(I'm out for the day, so if anyone happens to approve this and feel like 
pressing the "squash and merge" button that'd be swell, otherwise I'll get to 
it when I have a chance/tomorrow)

https://github.com/llvm/llvm-project/pull/96699
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to