[PATCH] D36110: [COFF, ARM64] Add MS builtins __dmb, __dsb, __isb

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 13:40:58 PDT 2017


mstorsjo added a comment.

Out of curiousity (I'm not familiar enough with this aspect of LLVM internals) - what practical difference does this make - wouldn't the existing llvm level intrinsic be usable as such already? (I do see that the same is done for `__builtin_arm_dmb` and `__dmb` etc for ARM already, so I'm sure it's right - just curious where it makes a difference.)


Repository:
  rL LLVM

https://reviews.llvm.org/D36110





More information about the llvm-commits mailing list