[PATCH] D99952: [X86] Initialize TargetOptions::StackProtectorGuardOffset member to its default value

[LemonBoy via Phabricator via llvm-commits]

LemonBoy added inline comments.


================
Comment at: llvm/include/llvm/Target/TargetOptions.h:334
     /// Stack protector guard offset to use.
-    unsigned StackProtectorGuardOffset : 32;
+    unsigned StackProtectorGuardOffset = (unsigned)-1;
 
----------------
nickdesaulniers wrote:
> Is the cast necessary?
Not really, unless `-Wsign-conversion` or equivalent options are passed.
I'm partial to keeping the explicit cast, if there are strong reasons to remove it I'll get rid of it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99952