[PATCH] D40885: [NFC] Refactor SafepointIRVerifier

Daniil Suchkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 03:10:35 PST 2017


DaniilSuchkov added inline comments.


================
Comment at: lib/IR/SafepointIRVerifier.cpp:140
 
+using AvailableValueSet = DenseSet<const Value *>;
+
----------------
mkazantsev wrote:
> Please, check it in as separate NFC. This is a big patch, and it is hard to follow what is going on while you're mixing refactoring and functional changes.
Nothing here is a functional change but I agree that this change should be a separate patch. 
https://reviews.llvm.org/D41002


https://reviews.llvm.org/D40885





More information about the llvm-commits mailing list