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

[LemonBoy via Phabricator via llvm-commits]

LemonBoy marked 2 inline comments as done.
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;
 
----------------
craig.topper wrote:
> 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?
Good call, thanks.


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

https://reviews.llvm.org/D99952