[PATCH] D59827: [slh] x86 impl of ARM instrinsic for SLH

Heejin Ahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 26 10:56:02 PDT 2019


aheejin added inline comments.


================
Comment at: clang/test/Preprocessor/init.c:9678
 // WEBASSEMBLY-NEXT:#define __GXX_ABI_VERSION 1002
+// WEBASSEMBLY-NEXT: #define __HAVE_SPECULATION_SAFE_VALUE 1
 // WEBASSEMBLY32-NEXT:#define __ILP32__ 1
----------------
Nit: Remove the whitespace to be consistent with adjacent lines? (I think having a whitespace is better in general though)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59827





More information about the cfe-commits mailing list