[PATCH] D28075: MergeFunctions: Preserve debug info in thunks, under option -mergefunc-preserve-debug-info
Anmol P. Paralkar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 11:12:24 PST 2017
appcs added a comment.
Thank you, everybody, for the valuable inputs.
I understand the the codegen consistency [-g, -g0] requirement; I agree.
Would it be fair to say that -mergefunc-preserve-debug-info is more suited to being placed under –Og?
I did not follow the "embed as a function attribute" part; please could you elaborate?
I will submit a subsequent patch (upon this current base patch) that implements an API to enable/disable this behaviour
(-mergefunc-preserve-debug-info) in the near future. Would that be alright?
Is the current base patch (Diff 84419) OK to be committed?
More information about the llvm-commits