[PATCH] D158137: Change -ffp-model= related warn_drv_overriding_flag_option to warn_drv_overriding_option

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 16 22:20:42 PDT 2023 

MaskRay added a comment.

In D158137#4594181 <https://reviews.llvm.org/D158137#4594181>, @dexonsmith wrote:

> LGTM.
>
> Perhaps as a follow-up, rename warn_drv_overriding_flag_option to have “t” in it?

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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158137/new/

https://reviews.llvm.org/D158137

More information about the cfe-commits mailing list