On Wed, Sep 20, 2017 at 11:20 AM, Tsimbalist, Igor V
<igor.v.tsimbal...@intel.com> wrote:
> Uros, could you please review this patch as it's a part of x86 specific
> changes you have reviewed already.
Please proofread and spell-check the text. There are grammatical errors, e.g.:
+When compiled with the @option{-finstrument-control-flow} option
+the ENDBR instruction also inserted at addresses to which indirect
and typos, such as:
+control-flow trasfer can happen.
English is not my tongue, but these two popped out immediately.
Other than that,
-a program security by checking a target address of control-flow
+a program security by checking target addresses of control-flow
Spurious change.
+@option{mshstk} options. Each of these options enables coresponding
Missing dash, and a typo.
+This option instructs the compiler to use only indirect branch tracking
+(indirect calls and jumps) support from x86 Control-flow Enforcment
Typo.
+Technology (CET) implementation, see also @code{-mcet} option. The
+option has effect only if @code{-finstrument-control-flow} option is
+specified. The option @code{-mibt} is on by default when @code{-mcet}
+option is specified.
Please use @option{} for options.
+This option instructs compiler to use only shadow stack (return address
+tracking) support from x86 Control-flow Enforcment Technology (CET)
+implementation, see also @code{-mcet} option. The option has effect
+only if @code{-finstrument-control-flow} option is specified. The
+option @code{-mshstk} is on by default when @code{-mcet} option is
+specified.
Also in the above text.
Uros.
