[PATCH] D23890: Fix ArrayRef initializer_list Ctor Test

Erich Keane via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 14:46:00 PDT 2016


erichkeane marked an inline comment as done.
erichkeane added a comment.

> I'd probably just keep one of these two tests - they don't seem to add a lot separately. Probably just keep the "more realistic" version, I would think (but put it in the existing test).

> 

> But no big deal either way - just my 2c :)


I actually just realized that the ArgTest12 tests exactly the 'realistic' test anyway.  I've removed the realistic test, and left this as simply the fix for the UB.


Repository:
  rL LLVM

https://reviews.llvm.org/D23890





More information about the llvm-commits mailing list