[PATCH] D27991: Optimize objdump -objc-meta-data
Saleem Abdulrasool via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 3 11:31:11 PST 2017
compnerd requested changes to this revision.
compnerd added inline comments.
This revision now requires changes to proceed.
================
Comment at: lib/Object/MachOObjectFile.cpp:3079
+ assert(
+ (Opcodes.begin() == Other.Opcodes.begin() || Opcodes == Other.Opcodes) &&
+ "compare iterators of different files");
----------------
These changes are technically introducing UB. Comparing iterators across containers is UB. So, we need a member-wise comparison. You can put this under expensive checks if you want though.
https://reviews.llvm.org/D27991
More information about the llvm-commits
mailing list