[PATCH] D79655: [WebAssembly] Handle exception specifications

Heejin Ahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 16 18:01:19 PDT 2020


aheejin marked an inline comment as done.
aheejin added inline comments.


================
Comment at: clang/test/CodeGenCXX/wasm-eh.cpp:387
 
+// RUN: %clang_cc1 %s -triple wasm32-unknown-unknown -fms-extensions -fexceptions -fcxx-exceptions -fwasm-exceptions -target-feature +exception-handling -emit-llvm -std=c++11 2>&1 | FileCheck %s --check-prefix=WARNING
+
----------------
thakis wrote:
> Ah, the problem is that this line is missing a -o flag and so it wrote a wasm-eh.ll file to the clang/test/CodeGenCXX directory, which on the next build was interpreted as a test without a run line. I'll land a temporary rm here to remove that file. Please fix this for the reland.
Oh I see... Thanks a lot for doing this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79655/new/

https://reviews.llvm.org/D79655





More information about the cfe-commits mailing list