[PATCH] [ADT][CMake][AutoConf] fail-fast iterators for DenseMap

Sanjoy Das sanjoy at playingwithpointers.com
Wed Mar 18 22:00:42 PDT 2015


When I was looking for a place to add a small blurb documenting this
new flag, I noticed this

``--disable-assertions``
    Builds LLVM with ``NDEBUG`` defined.  Changes the LLVM ABI.  Also available

in Packaging.rst.

If disabling assertions is *specified* to change the LLVM ABI then all
of this extra LLVM_ENABLE_ABI_BREAKING_CHANGES work is unneeded
complexity.  If not, then should this clause be removed from
Packaging.{rst|html}?

Thanks,
-- Sanjoy

On Wed, Mar 18, 2015 at 11:16 AM, Reid Kleckner <rnk at google.com> wrote:
> cmake changes lgtm
>
>
> http://reviews.llvm.org/D8351
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>



More information about the llvm-commits mailing list