[PATCH] D51317: Handle BumpPtrAllocator::Allocate(0) without undefined behavior

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 4 02:23:54 PDT 2018


hans added a comment.

In https://reviews.llvm.org/D51317#1221692, @brentdax wrote:

> @hans So do you think it's okay to go at this point, or do you want more benchmarking?


I'd like to see some kind of benchmark to verify this doesn't have negative performance impact.


Repository:
  rL LLVM

https://reviews.llvm.org/D51317





More information about the llvm-commits mailing list