[llvm] r231214 - Revert "[ADT] fail-fast iterators for DenseMap"
rfoos at codeaurora.org
Wed Mar 4 13:17:52 PST 2015
Should _DEBUG be undefined in a release build as well?
Sent from my Verizon Wireless 4G LTE smartphone
<div>-------- Original message --------</div><div>From: Zachary Turner <zturner at google.com> </div><div>Date:03/04/2015 2:02 PM (GMT-06:00) </div><div>To: Reid Kleckner <rnk at google.com> </div><div>Cc: Commit Messages and Patches for LLVM <llvm-commits at cs.uiuc.edu> </div><div>Subject: Re: [llvm] r231214 - Revert "[ADT] fail-fast iterators for DenseMap" </div><div>
</div>I can confirm that removing the ABI incompatibility fixes the issue for me
locally. I tested by commenting out the NDEBUG path in DebugEpochBase.h so
that it always used the debug path, and then rebuilding everything.
Can we change this to be _DEBUG instead of NDEBUG?
On Wed, Mar 4, 2015 at 11:42 AM Reid Kleckner <rnk at google.com> wrote:
> On Wed, Mar 4, 2015 at 9:43 AM, Zachary Turner <zturner at google.com> wrote:
>> How necessary to this change is the ABI incompatibility between the cases
>> where NDEBUG is and isn't defined? Is it possible to make them ABI
>> compatible while still keeping the meat of the change intact?
>> It doesn't really seem like a "nice" thing to do to require that people
>> who use LLVM as a library use the same compilation flags as LLVM.
> Early in the history of LLVM, we allowed NDEBUG to control struct layout
> and general ABI. Then Alp Toker came along and decided this was a Bad
> Thing. He audited the headers and removed it all. Now it's crept back in.
> In conclusion, what isn't tested will break.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits