Object of class Command contains various properties of a command to
execute, but output file was missed from them. This change adds this
property. It is required for reporting consumed time and memory implemented
in D78903 and may be used in other cases too.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Adding output files property is a prerequisite for D78903 (Add option -fproc-stat-report). Despite large size of the patch, the change is very simple: it only adds new state variable OutputFilenames initialized by the new parameter to the constructor of Command.
clang/lib/Driver/ToolChains/HIP.cpp | ||
---|---|---|
105 | I would suggest to use Args.MakeArgString(OutputFileName) instead of Output.c_str() to make sure it persists through the whole compilation. |
clang/lib/Driver/ToolChains/HIP.cpp | ||
---|---|---|
105 | Thank you for advice! |
clang/include/clang/Driver/Job.h | ||
---|---|---|
97 | Is this argument only used for reporting compile time and memory usage statistics? Can it have a default empty argument? There are lots of out-of-tree code creating 'Command'. Requiring this argument causes lots of hassles for them. |
Is this argument only used for reporting compile time and memory usage statistics?
Can it have a default empty argument?
There are lots of out-of-tree code creating 'Command'. Requiring this argument causes lots of hassles for them.