In these builds, the asserts() are completely compiled out of the code
leaving "End" unused. Directly accessing it, should not have a
performance impact, as it is just a data member.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@182634
91177308-0d34-0410-b5e6-
96231b3b80d8
}
void MCModule::map(MCAtom *NewAtom) {
}
void MCModule::map(MCAtom *NewAtom) {
- uint64_t Begin = NewAtom->Begin,
- End = NewAtom->End;
+ uint64_t Begin = NewAtom->Begin;
- assert(Begin < End && "Creating MCAtom with endpoints reversed?");
+ assert(Begin < NewAtom->End && "Creating MCAtom with endpoints reversed?");
// Check for atoms already covering this range.
AtomListTy::iterator I = std::lower_bound(atom_begin(), atom_end(),
Begin, AtomComp);
// Check for atoms already covering this range.
AtomListTy::iterator I = std::lower_bound(atom_begin(), atom_end(),
Begin, AtomComp);
- assert((I == atom_end() || (*I)->getBeginAddr() > End)
+ assert((I == atom_end() || (*I)->getBeginAddr() > NewAtom->End)
&& "Offset range already occupied!");
// Insert the new atom to the list.
&& "Offset range already occupied!");
// Insert the new atom to the list.