[PATCH] D87302: [IRSim][IROutliner] Adding DebugInfo handling for IR outlined functions.
Puyan Lotfi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 10 14:43:21 PDT 2021
plotfi added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/IROutliner.cpp:383
Group.OutlinedFunctionType, GlobalValue::InternalLinkage,
"outlined_ir_func_" + std::to_string(FunctionNameSuffix), M);
----------------
paquette wrote:
> dblaikie wrote:
> > Side note: This seems surprising to me - do any other passes create functions with plain names like this? I'd expect anything being created would use something like the C++ reserved identifier namespace (double leading underscore or leading underscore and uppercase).
> The MachineOutliner does this as well. It generally works okay because of the linkage type on the functions. It's not ideal though. IIRC there was a EuroLLVM talk planned prior to covid which had a better naming scheme.
>
> I think @plotfi was helping upstream some of those patches?
I was working on a pass for redoing address offset calculations done by GEPs so that more of the offset calculation will have increased IR and MIR similarity, but I don't think the size reduction was significant and there were also some weird pathological cases too. I gave up on it though.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87302/new/
https://reviews.llvm.org/D87302
More information about the llvm-commits
mailing list