[PATCH] D12347: [MC/AsmParser] Avoid setting MCSymbol.IsUsed

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 17:04:41 PDT 2015


vsk created this revision.
vsk added reviewers: rafael, ddunbar.
vsk added a subscriber: llvm-commits.

Avoid marking MCSymbols as used in MC/AsmParser.cpp when no uses exist.
    
This fixes a bug in parseAssignmentExpression() where we accidentally set IsUsed, thereby triggering:
    
    "invalid re-assignment of non-absolute variable"
    
on otherwise valid code. No other functionality is affected.

http://reviews.llvm.org/D12347

Files:
  lib/MC/MCParser/AsmParser.cpp
  test/MC/AsmParser/reassign.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12347.33154.patch
Type: text/x-patch
Size: 4848 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150826/b01528fd/attachment-0001.bin>


More information about the llvm-commits mailing list