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

[Craig Topper via Phabricator via llvm-commits]

craig.topper 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;
 
----------------
LemonBoy wrote:
> 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.
Use -1U?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99952