llvm/llvm-project

[meta] [debugify] fix -check-debugify failures

Open

#37,301 opened on Jun 27, 2018

View on GitHub
 (28 comments) (0 reactions) (0 assignees)C++ (26,378 stars) (10,782 forks)batch import
bugzilladebuginfogood first issuemetaissue

Description
Bugzilla Link 37953
Version trunk
OS All
Depends On llvm/llvm-project#37089 llvm/llvm-project#37303 llvm/llvm-project#37362 llvm/llvm-project#37290 llvm/llvm-project#37302 llvm/llvm-project#37304 llvm/llvm-project#37307 llvm/llvm-project#37308 llvm/llvm-project#37309 llvm/llvm-project#37310 llvm/llvm-project#37312 llvm/llvm-project#37314 llvm/llvm-project#37315 llvm/llvm-project#37371 llvm/llvm-project#37430 llvm/llvm-project#37431 llvm/llvm-project#37510 llvm/llvm-project#37511 llvm/llvm-project#37525 llvm/llvm-project#37526 llvm/llvm-project#37527
CC @adrian-prantl,@dwblaikie,@emaste,@flametop,@jmorse,@jdm,@jryans,@mikaelholmen,@pogo59,@Trass3r,@vedantk,@wjristow,@yurybura

Extended Description

This is a meta bug for tracking work related to fixing all known failures from debugify ( https://reviews.llvm.org/D40512 ).

The motivation for this is in order to provide the best possible user experience when debugging optimized code by ensuring that optimization passes preserve debug information wherever possible.

From Bug 37942, Comment 1:

An error report from -check-debugify should always be considered a compiler bug. It would be cumbersome if we couldn't rely on this -- so much so that we should consider relaxing -check-debugify rather than allowing it to emit false positives.

Contributor guide

Tech stack
cpp
Domain
backendtesting
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
5
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
over 1 week
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
C++LLVM IRdebug informationcompiler optimization passes
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
10
Research direction
This is a meta issue tracking many dependent issues (see Depends On). To contribute, first understand the debugify tool and its role in ensuring debug info preservation. Then examine the dependent issues to identify smaller sub tasks. The goal is to fix known failures in check debugify. No single code change is suggested here; review the linked issues and select one to resolve.