diff options
| author | Mircea Trofin <mtrofin@google.com> | 2025-09-12 15:05:16 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-09-12 15:05:16 -0700 |
| commit | 8f25ea2d73d9a4a64e7ab26e6b1d7a8f73605713 (patch) | |
| tree | d0f0123cc6e72ec22b6ebb7f23fae3b1944a215e /llvm/lib/Transforms/Utils/Local.cpp | |
| parent | 9e33997242800d49964bfbd056288cbb0cf073ed (diff) | |
| download | llvm-8f25ea2d73d9a4a64e7ab26e6b1d7a8f73605713.zip llvm-8f25ea2d73d9a4a64e7ab26e6b1d7a8f73605713.tar.gz llvm-8f25ea2d73d9a4a64e7ab26e6b1d7a8f73605713.tar.bz2 | |
[NFC] Leave a comment in `Local.cpp` about debug info & sample profiling (#155296)
Issue #152767
Diffstat (limited to 'llvm/lib/Transforms/Utils/Local.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/Local.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp index 2cfd70a..57dc1b3 100644 --- a/llvm/lib/Transforms/Utils/Local.cpp +++ b/llvm/lib/Transforms/Utils/Local.cpp @@ -3342,8 +3342,11 @@ void llvm::hoistAllInstructionsInto(BasicBlock *DomBlock, Instruction *InsertPt, // retain their original debug locations (DILocations) and debug intrinsic // instructions. // - // Doing so would degrade the debugging experience and adversely affect the - // accuracy of profiling information. + // Doing so would degrade the debugging experience. + // + // FIXME: Issue #152767: debug info should also be the same as the + // original branch, **if** the user explicitly indicated that (for sampling + // PGO) // // Currently, when hoisting the instructions, we take the following actions: // - Remove their debug intrinsic instructions. |