hans added a comment. > Thanks! I agree. d9ad0681fad9a98f43d9baddb95d505b37153c48 (2013) renamed > `warn_drv_overriding_t_option` to `warn_drv_overriding_flag_option`. > Perhaps the original name `warn_drv_overriding_t_option` should be restored.
That change also started using it for overriding `/MD` and `/MT`. I think the intention then was to make the flag more general, but it forgot to rename the flag spelling. Perhaps the fix is really to s/overriding-t-option/overriding-option/ ? (If we're concerned about the interface change, we could retain an alias with the old spelling.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158137/new/ https://reviews.llvm.org/D158137 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
