[llvm] r216363 - NVPTX: remove raw delete call

Chandler Carruth chandlerc at google.com
Mon Aug 25 09:37:49 PDT 2014


On Mon, Aug 25, 2014 at 9:32 AM, David Blaikie <dblaikie at gmail.com> wrote:

> The change from public to private makes this patch a bit harder to
> read (since the existing SmallVectors show up as -/+ lines when they
> were just moved) - might've been better in a separate patch.
>

Also, it should really have been reviewed by some of the maintainers of the
backend. There might be legitimate reasons that these were left public in
order to facilitate future work...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140825/f2fa1b7d/attachment.html>


More information about the llvm-commits mailing list