================
@@ -1796,16 +1796,21 @@ bool tools::addSanitizerRuntimes(const ToolChain &TC, 
const ArgList &Args,
     CmdArgs.push_back("--export-dynamic-symbol=__cfi_check");
 
   if (SanArgs.hasMemTag()) {
-    if (!TC.getTriple().isAndroid()) {
+    if (TC.getTriple().isWasm()) {
+    }
----------------
sbc100 wrote:

This empty block looks a little odd, perhaps this should instead be combined 
with the else below?  `else if (!TC.getTriple().isWasm())`?   

Also should this be wasm64?

https://github.com/llvm/llvm-project/pull/162972
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to