[llvm-commits] [llvm] r159785 - /llvm/trunk/include/llvm/MC/MCAssembler.h
Owen Anderson
resistor at mac.com
Thu Jul 5 15:30:43 PDT 2012
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);
}
More information about the llvm-commits
mailing list