[PATCH] D18808: Use the NoDebug emission kind to identify compile units that no debug info should be created from.
Duncan P. N. Exon Smith via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 5 21:35:14 PDT 2016
IR changes LGTM, if you add the missing verifier check (I'm surprised
this *ever* passed the verifier...)
Someone else will have to look at DwarfDebug.cpp.
> On 2016-Apr-05, at 16:55, Adrian Prantl <aprantl at apple.com> wrote:
> aprantl created this revision.
> aprantl added reviewers: dblaikie, echristo, dexonsmith.
> aprantl added subscribers: davide, llvm-commits, cfe-commits.
> aprantl set the repository for this revision to rL LLVM.
> As indicated in D18612, sample-based profiling and optimization remarks currently remove DICompileUnits from llvm.dbg.cu to suppress the emission of debug info from them. This is somewhat of a hack and only borderline legal IR.
> This patch uses the recently introduced NoDebug emission kind in DICompileUnit to achieve the same result without breaking the Verifier. A nice side-effect of this change is that it is now possible to combine NoDebug and regular compile units under LTO.
> For simplicity I combined the llvm and cfe patches into one review.
> rL LLVM
More information about the cfe-commits