<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle">P {
MARGIN-TOP: 0px; MARGIN-BOTTOM: 0px
}
</style>
</head>
<body fPStyle="1" ocsi="0">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">
<p>> Few comments: </p>
<div style="FONT-FAMILY: Times New Roman; COLOR: #000000; FONT-SIZE: 16px">
<div>
<div dir="ltr">
<div>> <br>
</div>
<div>
<div>> + DW_TAG_CXX_static_member = 0x104, // Tag for C++ static data member.</div>
</div>
<div>> <br>
</div>
<div>> there is no such tag that I've seen?</div>
<div> </div>
<div>This is in enum llvm_dwarf_constants, not enum dwarf_constants.</div>
<div>It's one of several mock tags defined for the metadata interface;</div>
<div>they are not defined by DWARF. (If the number of real DWARF tags</div>
<div>doubles, we'll have to renumber these to get out of the way.)</div>
<div> </div>
<div>> And you don't disassemble to it either.</div>
<div>Not sure what you mean here.</div>
<div><br>
> Also you don't use isStaticData()...</div>
<div>><br>
</div>
<div>> Perhaps there's some part of the patch I'm missing?</div>
<div> </div>
<div>See the Clang part of the patch, which does use isStaticDataMember().</div>
<div>--paulr</div>
<div> </div>
</div>
</div>
</div>
</div>
</body>
</html>