debugging hacks, not to be merged

Merge branch 'pmachata/dwarflint-smartforms' into dwarf

dwarflint: Drop read_ctx_read_form in favor of read_generic_value

dwarflint: Fix a typo

dwarflint: Introduce read_generic_value into checked_read
- the hope being several reading operations can be expressed in terms of
  read_generic_value

dwarflint: move read_sc_value to checked_read module

dwarflint: checked_read takes const where

Use std:tr1::hash<std::string> where possible.

* c++/subr.hh (string_hash_combine): Removed.
  (hash<std::string>): Extend std::tr1::hash<std::string>.
* c++/dwarf_data (value_string): typedef std::tr1::hash<std::string>
  hasher.

dwarflint: Update TODO

dwarflint: Change reloc_target signature
- leave the body as is for the time being

dwarflint: Streamlining code
- Extract checks common to direct and indirect forms to
  check_debug_abbrev::check_form.
- Reading of indirect form is now done via a new function read_sc_value
  in check_debug_info.cc.  That function is also called for reading
  normal values of classes sc_value and sc_block

dwarflint: Use a cl_indirect test instead of testing DW_FORM_indirect

dwarflint: Remove duplicate testing for allowed forms in check_debug_abbrev

dwarflint: Add DW_FORM_indirect to DWARF 2
- add cl_indirect to facilitate the indirect form.  Make this the implicit
  class of all attributes.

dwarflint: Retrofit cl_lineptr and cl_macptr to DWARF 2
- which fixes the failing test case

dwarflint: Retrofit proper marking of exprloc and loclistptr to earlier DWARFs
- as a consequence, is_location_attrib is gone
- in check_debug_abbrev, we now check for appropriateness all form/attribute
  pairs, not just those that are location attribs
- too-wide references are now checked in one place for all reference types
- pointer checking now actually looks sane--the maze of DW_FORM_/DW_AT_
  switches was replaced by a straighforward switch on ver->form_class
- this introduces a testcase failure
- and similar work needs to be done for other cl_*ref classes

dwarflist: Fix dwarf_version::form_class

dwarflint: Nits

dwarflint: Fix classes for DW_AT_accessibility, _abstract_origin in DWARF2

dwarflint: Support MIPS extensions
- and unconditionally return it in request to dwarf_version::get

dwarflint: Remove part of the loclist switching
- use the new articulated forms/attributes instead
- the goal is to make most of the code like this

dwarflint: Add dwarf_version::form_class, ~::ambiguous_class, drop form::cls
- And use that in dwarf_3 to properly support DW_AT_data_member_location

dwarflint: Add form::cls to get the class of form in context of given attribute

dwarflint: Rename variable

dwarfint: Nits

dwarflint: Devirtualize form and attribute

dwarflint: Drop duplicate code

dwarflint: Shuffle code

dwarflint: Move more code from dwarf_version.hh to .cc
- And drop selwidth_form, which was overly general for the two uses that we
  have.  Implement corresponding classes directly.

dwarflint: Header fixlets

dwarflint: Split the dwarf_version.cc into modules by DWARF version

dwarflint: Rename tables.* to dwarf_version.*

dwarflint: Introduce dwarf_version::extend
- Drop support for parental version in std_dwarf, implement the same via
  the extension mechanism
- Drop the huge #if 0 that has been there for about a year or so

dwarflint: Make DWARF attributes objects, too

dwarflint: Use form::storage_class and checked_read_leb128 in check_debug_info

dwarflint: Add form::storage_class

dwarflint: Add checked_read_leb128
- this is similar to read_ctx_read_var
- also drop a bunch of C-isms

dwarflint: Minor adjustments in tables.cc
- After some experimenting with different ways of defining form classes,
  I'm back to the original code.

dwarflint: Remove dwarf_version::form_width
- and a bunch of changes in dwarf_version internals

dwarflint: Make a bunch of internal types related to dwarf_version public

dwarflint: Use dwarf_version::form_width in check_debug_info

dwarflint: Revamping dwarf tables
- forms are now objects.  The idea being they know how big they are, how
  to relocate them, perhaps they might know how to extract value from
  stream.  The goal here is to concetrate all the form-related knowledge
  to a sigle place.  It now passes the suite, but is not really done yet.

dwarflint: Fix handling of indirect forms in check_debug_info
- and commit a couple other nits

dwarflint: Use wrapper to print check descriptions
- this should make it easier to write them
- also use argp to print out properly formatted help for each check.  Drop
  a bunch of code that was necessary to do things the old way

dwarflint: Avoid constructing temporary std::string in line wrapper

dwarflint: Fix off-by-one in spaces

dwarflint: Add a function to format N spaces

dwarflint: Fix wrapping long words on indented lines

dwarflint: Add line wrapper
- it's not currently used, but we will use this for formatting of check
  documentation and options

dwarflint: Check coverage of arange vs. CUs as well as the other way around

dwarflint: Test case for aranges that terminate early

dwarflint: Promote duplicate-arange-for-CPU to error

dwarflint: Check for CUs that miss arange table
+testcase

dwarflint: check_range_out_of_scope should run even on absent .debug_loc
- the actual fix is removing the forced lint.check <check_debug_loc>.
  The lowlevel_checks pass takes care of validating any present sections.
  In this case .debug_loc isn't there, yet the check needs to run.
- and add a test case for this.

dwarflint: --list-checks now lists options associated with check passes

dwarflint: Rename local options to include the check pass prefix
- The original intention was that this is added automatically, but perhaps
  the better way to handle it is to make it easy to pick whatever name one
  wishes.  Short options will share namespace anyway, so one has to be
  careful with their naming anyway.

dwarflint: Recognize options local to check

dwarflint: Only toplevel option pack should have description

dwarflint: Better description of options in locstats

dwarflint: Show check-local options in --help
- They are still not properly recognized
- And name mangling isn't implemented

dwarflint: Don't assume all options are global
- currently this breaks dwarflint, because per-check options are not yet
  processed.  Options that belonged to global group before still work and
  are correctly registered as globals, but e.g. -i or the suite of options
  at locstats aren't recognized.

dwarflint: Get rid of low.h/low.c
- and rename misc.h to misc.hh

dwarflint: Getting rid of compat C bindings

dwarflint: Move much of the rest of low.h to where it belongs

dwarflint: Move abbrev-related data structures to check_debug_abbrev

dwarflint: More moving stuff around

dwarflint: New module checked_read
- a bunch of functions that I couldn't place just right are there

dwarflint: Drop unused declarations

dwarflint: check_range_relocations should rather be in check_debug_loc_range

dwarflint: Move read_rel and check_range_relocations to reloc module

dwarflint: Move check_aranges_structural to check_debug_aranges

dwarflint: Move check_zero_padding to read_ctx module
- and promote the reporting part back to callers

dwarflint: Move read_address_size to read_ctx module
- and shuffle arguments around so it makes sense

dwarflint: Move read_size_extra to readctx module

dwarflint: Add DIE classification categories "mutable" and "immutable"
- however I guess the better thing to do would be to dump how big part
  of a scope is mutable vs. immutable, or something like that

dwarflint: Fix classification of no_coverage DIEs

dwarflint: Implement reading DW_FORM_ref_addr forms
- Patch by Jakub Jelinek

In dwarf-opcodes.h, add DW_OP_GNU_implicit_pointer and fix DW_OP_call_ref
- patch by Jakub Jelinek

dwarflint: Add a possibility to distinguish sharp 0 coverage to locstats pass

dwarflint: Add a description of locstats

dwarflint: Add an option to recognize inlined subroutines

dwarflint: Add more options for ignoring and dumping dies to locstat
- these are probably mostly useful for myself as I'm trying to find out
  what's really missing the info and what doesn't need one it the first
  place, but why not allow others to do the tinkering too...

dwarflint: Exclude any location list entries whose expression is empty

dwarflint: In all_dies_iterator, replace parent () with stack ()
- use it in locstats to find the nearest parental DIE with non-empty ranges

dwarflint: First stab at location/value stats
- things will change, some parts of the proposal are not yet implemented

dwarflint: Publish C dwarf pointer in high-level checks

dwarflint: Add the ability to query the die parent in all_dies_iterator

dwarflint: Only warn about unused rules if we had a chance to match them

dwarflint: Coding style nits

dwarflint: Warn about --check rules that never match

dwarflint: Extract checkrule into module of its own

Merge branch 'dwarf' of ssh://git.fedorahosted.org/git/elfutils into dwarf

Only run low-level checks in dwarflint self-check

The reason being that high-level checks do and always have emitted a lot
of output, mainly for big c++ binaries.  The low-level checks should
generally pass, and even if there are nits here and there, they should be
minor and shouldn't flood the terminal.

Merge commit 'elfutils-0.149' into dwarf

dwarflint: Make it possible to define relevant options directly at checks

dwarflint: And add the description

dwarflint: Add a test for nodebug check

dwarflint: Check for missind debuginfo sections separately

dwarflint: Fix test case
- now that we run the low-level checks as prerequisite of high-level ones,
  we also get low-level errors in testsuite passes.  One more reason to
  implement some sensible message filtering command line options.

dwarflint: Move secinfo_map initiazitaion into ctor, avoid macros
- and reindent two statements below where it's instantiated
- and add a documentation to the section_name array