[PATCH] D49234: [DebugInfo] Refactor DWARFDie::iterator to prevent UB

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 23 10:24:01 PDT 2018

Previous message: [llvm] r337711 - [ARM] Follow-up to r337709.
Next message: [PATCH] D49680: [ADT] Replace std::isprint by llvm::isPrint.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

JDevlieghere added a comment.

I've put up https://reviews.llvm.org/D49679 as an alternative to this patch.  Personally I don't like the caveat that you can't store the DWARFDie the iterators point to, but on the other hand I don't like returning by value either. Please let me know which you prefer.


https://reviews.llvm.org/D49234

Previous message: [llvm] r337711 - [ARM] Follow-up to r337709.
Next message: [PATCH] D49680: [ADT] Replace std::isprint by llvm::isPrint.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list