[PATCH] D26144: DebugInfo: make DW_TAG_atomic_type valid

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 08:33:57 PST 2016


On Thu, Dec 8, 2016 at 11:00 PM, Victor Leschuk <vleschuk at accesssoftek.com>
wrote:

> Hello Reid,
>
> On 10/31/2016 07:06 PM, Reid Kleckner wrote:
>
>> rnk added inline comments.
>>
>>
>> ================
>> Comment at: lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:1412
>> +    case dwarf::DW_TAG_atomic_type:
>> +      Mods |= ModifierOptions::Atomic;
>> +      break;
>> ----------------
>> Let's ignore _Atomic qualifiers for now. I emailed Dave Bartolomeo about
>> reserving that bit, and eventually getting MS debuggers to print the
>> _Atomic qualifier, but until we hear back, I don't want to start emitting
>> it. Leave a FIXME to follow up on it.
>>
> Have you received any reply from Dave regarding this CodeView bit?


Not yet. I pinged him again and asked for a more appropriate contact if one
exists.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161209/859ebfc0/attachment.html>


More information about the llvm-commits mailing list