<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Feb 7, 2017 at 11:01 AM Amjad Aboud via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aaboud added a comment.<br class="gmail_msg">
<br class="gmail_msg">
In <a href="https://reviews.llvm.org/D16135#669416" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D16135#669416</a>, @aprantl wrote:<br class="gmail_msg">
<br class="gmail_msg">
> In <a href="https://reviews.llvm.org/D16135#669045" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D16135#669045</a>, @aaboud wrote:<br class="gmail_msg">
><br class="gmail_msg">
> > Addressed Adrian last comments.<br class="gmail_msg">
> >  Added a LIT tests that covers all the macro kinds:<br class="gmail_msg">
> ><br class="gmail_msg">
> > 1. built-in (define)<br class="gmail_msg">
> > 2. command-line (define, include, undef)<br class="gmail_msg">
> > 3. main source (define, include, undef) Checked the above with and without PCH include.<br class="gmail_msg">
> ><br class="gmail_msg">
> >   Notice, that current implementation does not support debug info for macro definition and inclusion generated during the PCH file creation. To support that, we need to extend the PCH format to preserve this information. If you believe this feature is important, I will open a bugzilla ticket and we can handle it separately.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> That would be a good idea and important to fix. I don't think we want the the generated code (or the debug info for that matter) to change just because the user decides to switch to PCH. Ideally turning on PCH should be transparent.<br class="gmail_msg">
><br class="gmail_msg">
> I have one final question: What's the impact of this change on, e.g., building clang?<br class="gmail_msg">
><br class="gmail_msg">
> - How much does the build directory grow?<br class="gmail_msg">
> - Is there any noticeable compile time regression?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
I will run self-build and come back with answers for these questions.<br class="gmail_msg">
I am just wondering, if I build in debug mode, will it use the flag "-fstandalone-debug"?<br class="gmail_msg"></blockquote><div><br></div><div>I'm not sure it makes sense to motivate this feature with this flag. I'm going to assume Darwin and FreeBSD don't want to implicitly pick up this feature because of the default on their platforms which is really to address a particular limitation in LLDB due to some other debug info.<br><br>So this probably needs a separate flag.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This flag is set by default only for DarwinClang and FreeBSD, is there a way to assure it will be used with self-build?<br class="gmail_msg"></blockquote><div><br></div><div>You can add flags to your build by modifying your CMakeCache.txt file and putting them int he CMAKE_CXX_FLAGS/CMAKE_C_FLAGS values (or the specific ones for teh build mode you'd like them to go with, like the DEBUG build, most likely)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Notice that without this flag, there will be no debug info macro generated.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D16135" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D16135</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>