[clang] [Driver] Clean up fp-contract handling in clang driver (PR #91271)

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Fri May 10 10:47:31 PDT 2024


================
@@ -2751,6 +2751,7 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D,
   // If one wasn't given by the user, don't pass it here.
   StringRef FPContract;
   StringRef LastSeenFfpContractOption;
+  StringRef LastFpContractOverrideOption;
----------------
MaskRay wrote:

`Overridden` seems more appropriate, but the variable name will become too long.
I wonder whether `Last` can be removed. It's clear from the type that this can only hold one option name.

https://github.com/llvm/llvm-project/pull/91271


More information about the cfe-commits mailing list