[llvm-commits] [llvm] r159785 - /llvm/trunk/include/llvm/MC/MCAssembler.h
Rafael EspĂndola
rafael.espindola at gmail.com
Thu Jul 5 18:38:29 PDT 2012
Test case?
On Jul 5, 2012 6:31 PM, "Owen Anderson" <resistor at mac.com> wrote:
> Author: resistor
> Date: Thu Jul 5 17:30:42 2012
> New Revision: 159785
>
> URL: http://llvm.org/viewvc/llvm-project?rev=159785&view=rev
> Log:
> Fix an overzealous assertion. It is legitimate for a target to have
> multiple fixups on a single instruction that target the same byte, so long
> as their bit-offsets are coordinates appropriately.
>
> Modified:
> llvm/trunk/include/llvm/MC/MCAssembler.h
>
> Modified: llvm/trunk/include/llvm/MC/MCAssembler.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCAssembler.h?rev=159785&r1=159784&r2=159785&view=diff
>
> ==============================================================================
> --- llvm/trunk/include/llvm/MC/MCAssembler.h (original)
> +++ llvm/trunk/include/llvm/MC/MCAssembler.h Thu Jul 5 17:30:42 2012
> @@ -130,7 +130,7 @@
>
> void addFixup(MCFixup Fixup) {
> // Enforce invariant that fixups are in offset order.
> - assert((Fixups.empty() || Fixup.getOffset() >
> Fixups.back().getOffset()) &&
> + assert((Fixups.empty() || Fixup.getOffset() >=
> Fixups.back().getOffset()) &&
> "Fixups must be added in order!");
> Fixups.push_back(Fixup);
> }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120705/0147067f/attachment.html>
More information about the llvm-commits
mailing list