From: Eric Christopher Date: Wed, 7 Aug 2013 18:51:09 +0000 (+0000) Subject: Move assert above first use of variable that we'd be asserting on. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=ee0dcf4b3ef6a0dbc03ceb96ca1bb6eef8b26e70;p=oota-llvm.git Move assert above first use of variable that we'd be asserting on. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187899 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/MC/MCAssembler.cpp b/lib/MC/MCAssembler.cpp index 24f4b8e635e..68111f187d3 100644 --- a/lib/MC/MCAssembler.cpp +++ b/lib/MC/MCAssembler.cpp @@ -580,10 +580,10 @@ static void writeFragment(const MCAssembler &Asm, const MCAsmLayout &Layout, case MCFragment::FT_Align: { ++stats::EmittedAlignFragments; const MCAlignFragment &AF = cast(F); - uint64_t Count = FragmentSize / AF.getValueSize(); - assert(AF.getValueSize() && "Invalid virtual align in concrete fragment!"); + uint64_t Count = FragmentSize / AF.getValueSize(); + // FIXME: This error shouldn't actually occur (the front end should emit // multiple .align directives to enforce the semantics it wants), but is // severe enough that we want to report it. How to handle this?