r265038 - Diagnostics: remove dodgy handler for bitcode inlineasm diagnostics.
Tim Northover via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 1 12:04:07 PDT 2016
> On 31 Mar 2016, at 16:51, Steven Wu <stevenwu at apple.com> wrote:
>
> The original handler is not there to workaround a crash. It is to avoid generate a crash report (report_fatal_error) when the input bitcode contains invalid assembly.
Ah, I see now. That test needs tweaking too, since it doesn't actually produce the crash report even after my change (probably because it's a %clang_cc1 test).
How does the patch attached here look? I originally tried reusing the real InlineAsmDiagHandler but that was disastrous (it needs a real AST I think, which obviously doesn't exist when compiling a .bc file).
Cheers.
Tim.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: asm-diagnostics.diff
Type: application/octet-stream
Size: 2589 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160401/04927e20/attachment.obj>
More information about the cfe-commits
mailing list