Closed Bug 512887 Opened 15 years ago Closed 15 years ago

misc hydra fixes for callgraph work

Categories

(Developer Infrastructure :: Source Code Analysis, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dwitte, Unassigned)

References

Details

Attachments

(1 file)

Attached patch patch v1Splinter Review
This rolls up a bunch of fixes.

* make hasDefaults property work (bug 510212)
* add resolve_path() utility function for resolving symlinks (to be removed later)
* add/tweak some macro stuff in gcc_compat.js
* fix type_string() to work with ptr-to-members and C++/C code in our tree, to order qualifiers (const etc) more consistently with g++, to work with vectors, and consistently throw if it gets confused - so we can spot errors easier. (I'm basically relying on this function being sane for serializing callgraph method types.)
* random fix to dehydra_attachClassStuff() to make it not die on classes with no bases
* tests to tickle most codepaths in type_string()
Attachment #396947 - Flags: review?(tglek)
Er, wrong component. ;)
Component: DOM: Abstract Schemas → Rewriting and Analysis
QA Contact: general → rewriting-and-analysis
Comment on attachment 396947 [details] [diff] [review]
patch v1


>+function DECL_IS_BUILTIN(decl) {
>+  return DECL_SOURCE_LOCATION(decl).line == 0;
>+}

that's so not what tree.h does. These should be identical to gcc, otherwise it gets tricky to debug what's going wrong.

Seems ok otherwise, r+ with above fixed.
Attachment #396947 - Flags: review?(tglek) → review+
Oh, does gcc/g++ #define USE_MAPPED_LOCATION?
http://hg.mozilla.org/rewriting-and-analysis/dehydra/rev/6308679eb663
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: