oren_ben_simhon updated this revision to Diff 127008.
oren_ben_simhon added a comment.
Implemented all comments posted until 12/14 (Thanks Peter)
Repository:
rL LLVM
https://reviews.llvm.org/D40478
Files:
include/clang/Driver/Options.td
include/clang/Frontend/CodeGenOptions.def
craig.topper added inline comments.
Comment at: include/clang/Driver/Options.td:1035
HelpText<"Instrument function entry only, after inlining, without arguments
to the instrumentation call">;
-
+def finstrument_control_flow : Flag<["-"], "finstrument-control-flow">,
+
pcc added inline comments.
Comment at: include/clang/Driver/Options.td:1035
HelpText<"Instrument function entry only, after inlining, without arguments
to the instrumentation call">;
-
+def finstrument_control_flow : Flag<["-"], "finstrument-control-flow">,
+ Group,
craig.topper added a comment.
-mibt isn't required for inline assembly, There's no AssemblePredicate defined
with HasIBT in X86InstrInfo.td. We don't normally do fine grained assembler
feature enabling so I believe we should always support it.
Repository:
rL LLVM
