thurstond wrote: > This looks good but I wonder if we should also have default initialisers in > LifetimeSafetyOpts as well to avoid problems like this in the future.
This issue only occurs in the test. Normally LifetimeSafetyOpts are constructed via GetLifetimeSafetyOpts, which sets the values appropriately: https://github.com/llvm/llvm-project/blob/f7ce52f0f4ec943bfdc749bc67de00a1e3ece6bc/clang/lib/Sema/SemaLifetimeSafety.h#L120-L131 https://github.com/llvm/llvm-project/pull/221843 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
