<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 31 January 2014 17:53, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>I think you want 'Args.hasFlag(options::OPT_fno_exceptions, options::OPT_fexceptions, false)' in order to make sure that 'clang -fno-exceptions -fexceptions' produces unwind tables.</div>
</div></div></div></blockquote><div><br></div><div>Yes, I changed that already. Thanks!</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div>I also seem to recall that there are major problems with -backend-option if you want to run Clang on multiple TUs in the same process.  It causes us to call llvm::cl::ParseCommandLineOptions twice, which isn't allowed, and bad things happen.  It doesn't have to be fixed now, but long term it needs to go away if anyone wants to, say, run a Clang tool over code that normally uses -fno-exceptions on ARM.<br>
</div></div></div></div></blockquote><div><br></div><div>There were some discussions on multiple calls to the driver on other threads. I have to say I also get very confused on which way Clang is using which arguments. For this patch, I just used what was already there, so it'd be easier to whomever refactors the driver's code to do them all at once.</div>
<div><br></div><div>cheers,</div><div>--renato</div></div><br></div></div>