Commit f6d98429 authored by serge-sans-paille's avatar serge-sans-paille
Browse files

Revert "Support -fstack-clash-protection for x86" 

This reverts commit 39f50da2.

The -fstack-clash-protection is being passed to the linker too, which
is not intended.

Reverting and fixing that in a later commit.
parent c4f0f8ec

clang/docs/ClangCommandLineReference.rst

+0 −4
Original line number Diff line number Diff line
@@ -1917,10 +1917,6 @@ Use a strong heuristic to apply stack protectors to functions 


Emit section containing metadata on function stack sizes



.. option:: -fstack-clash-protection, -fno-stack-clash-protection



Instrument stack allocation to prevent stack clash attacks (x86, non-Windows only).



.. option:: -fstandalone-debug, -fno-limit-debug-info, -fno-standalone-debug



Emit full debug info for all types used by the program

clang/docs/ReleaseNotes.rst

+0 −4
Original line number Diff line number Diff line
@@ -61,10 +61,6 @@ New Compiler Flags 
------------------





- -fstack-clash-protection will provide a protection against the stack clash

  attack for x86 architecture through automatic probing of each page of

  allocated stack.



Deprecated Compiler Flags

-------------------------

clang/include/clang/Basic/CodeGenOptions.def

+0 −1
Original line number Diff line number Diff line
@@ -150,7 +150,6 @@ CODEGENOPT(NoWarn , 1, 0) ///< Set when -Wa,--no-warn is enabled. 
CODEGENOPT(EnableSegmentedStacks , 1, 0) ///< Set when -fsplit-stack is enabled.

CODEGENOPT(NoInlineLineTables, 1, 0) ///< Whether debug info should contain

                                     ///< inline line tables.

CODEGENOPT(StackClashProtector, 1, 0) ///< Set when -fstack-clash-protection is enabled.

CODEGENOPT(NoImplicitFloat   , 1, 0) ///< Set when -mno-implicit-float is enabled.

CODEGENOPT(NoInfsFPMath      , 1, 0) ///< Assume FP arguments, results not +-Inf.

CODEGENOPT(NoSignedZeros     , 1, 0) ///< Allow ignoring the signedness of FP zero

clang/include/clang/Basic/DiagnosticCommonKinds.td

+0 −4
Original line number Diff line number Diff line
@@ -239,10 +239,6 @@ def note_invalid_subexpr_in_const_expr : Note< 
let CategoryName = "Inline Assembly Issue" in {

  def err_asm_invalid_type_in_input : Error<

    "invalid type %0 in asm input for constraint '%1'">;



  def warn_stack_clash_protection_inline_asm : Warning<

    "Unable to protect inline asm that clobbers stack pointer against stack clash">,

    InGroup<DiagGroup<"stack-protector">>;

}



// Sema && Serialization

clang/include/clang/Basic/TargetInfo.h

+0 −2
Original line number Diff line number Diff line
@@ -816,8 +816,6 @@ public: 
  StringRef getNormalizedGCCRegisterName(StringRef Name,

                                         bool ReturnCanonical = false) const;



  virtual bool isSPRegName(StringRef) const { return false; }



  /// Extracts a register from the passed constraint (if it is a

  /// single-register constraint) and the asm label expression related to a

  /// variable in the input or output list of an inline asm statement.