diff options
author | Mircea Trofin <mtrofin@google.com> | 2018-12-21 22:48:50 +0000 |
---|---|---|
committer | Mircea Trofin <mtrofin@google.com> | 2018-12-21 22:48:50 +0000 |
commit | b53eeb6f4c824fcdfcb93dd9325a603c005660a7 (patch) | |
tree | 584b0989c4d54efc27f9a41e0c3430741f39fd7d /llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp | |
parent | 8d20cfdfc6c26533a87c525cb6e8dbc50a65e036 (diff) | |
download | llvm-b53eeb6f4c824fcdfcb93dd9325a603c005660a7.zip llvm-b53eeb6f4c824fcdfcb93dd9325a603c005660a7.tar.gz llvm-b53eeb6f4c824fcdfcb93dd9325a603c005660a7.tar.bz2 |
[llvm] API for encoding/decoding DWARF discriminators.
Summary:
Added a pair of APIs for encoding/decoding the 3 components of a DWARF discriminator described in http://lists.llvm.org/pipermail/llvm-dev/2016-October/106532.html: the base discriminator, the duplication factor (useful in profile-guided optimization) and the copy index (used to identify copies of code in cases like loop unrolling)
The encoding packs 3 unsigned values in 32 bits. This CL addresses 2 issues:
- communicates overflow back to the user
- supports encoding all 3 components together. Current APIs assume a sequencing of events. For example, creating a new discriminator based on an existing one by changing the base discriminator was not supported.
Reviewers: davidxl, danielcdh, wmi, dblaikie
Reviewed By: dblaikie
Subscribers: zzheng, dmgreen, aprantl, JDevlieghere, llvm-commits
Differential Revision: https://reviews.llvm.org/D55681
llvm-svn: 349973
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp index b5d80f6..e267626 100644 --- a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp +++ b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp @@ -300,8 +300,15 @@ LoopUnrollResult llvm::UnrollAndJamLoop( for (BasicBlock *BB : L->getBlocks()) for (Instruction &I : *BB) if (!isa<DbgInfoIntrinsic>(&I)) - if (const DILocation *DIL = I.getDebugLoc()) - I.setDebugLoc(DIL->cloneWithDuplicationFactor(Count)); + if (const DILocation *DIL = I.getDebugLoc()) { + auto NewDIL = DIL->cloneWithDuplicationFactor(Count); + if (NewDIL) + I.setDebugLoc(NewDIL.getValue()); + else + LLVM_DEBUG(dbgs() + << "Failed to create new discriminator: " + << DIL->getFilename() << " Line: " << DIL->getLine()); + } // Copy all blocks for (unsigned It = 1; It != Count; ++It) { |