zimirza wrote:

> > `localtime_s` was removed from #110363, and will be added to a new pull 
> > request and `ctime_s` has not been merged yet (#110676). It might be a good 
> > wait before merging this pull request.
> 
> I think it is better to remove `localtime_s` and `ctime_s` from this pull 
> request in order to keep changes small. These will be added to a new pull 
> request when the implementations are merged.

`localtime_s` and `ctime_s` is currently available in a separate branch: 
https://github.com/zimirza/llvm-project/tree/add_time_to_clang_tidy_with_ctime_s_and_localtime_s

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

Reply via email to