djtodoro marked 2 inline comments as done.
djtodoro added a comment.
@probinson @aprantl Thanks a lot for your comments!
Let's clarify some things. I'm sorry about the confusion.
Initial patch for the functionality can be restricted by this option (like we
pushed here), since **LLDB** doesn't
aprantl added a comment.
And if we plan to enable it by default, too, perhaps not adding a driver option
(CC1 only) is preferable, since we need to maintain compatibility with driver
options indefinitely.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58033/new/
probinson added a comment.
I guess I'm not clear what your final goal is for the option. Keep it, even
though GCC doesn't have one like it? Eliminate it? Please clearly state what
you intend to have in the end, and what you might have in the short term in
case that is different.
djtodoro marked an inline comment as done.
djtodoro added inline comments.
Comment at: include/clang/Driver/Options.td:919
HelpText<"Do not use jump tables for lowering switches">;
+def emit_param_entry_values : Joined<["-"], "femit-param-entry-values">,
+
aprantl added inline comments.
Comment at: include/clang/Driver/Options.td:919
HelpText<"Do not use jump tables for lowering switches">;
+def emit_param_entry_values : Joined<["-"], "femit-param-entry-values">,
+ Group,
I assume that
djtodoro updated this revision to Diff 195188.
djtodoro edited the summary of this revision.
djtodoro added a comment.
-Rebase
-Add all_call_sites flag in the case of GNU extensions
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58033/new/
https://reviews.llvm.org/D58033
Files:
