[PATCH] D123439: [VE][compiler-rt] Correct location of VE support in clear_cache function, NFC

[Brad Smith via Phabricator via llvm-commits]

brad created this revision.
brad added a reviewer: simoll.
brad added a project: LLVM.
Herald added a subscriber: dberris.
Herald added a project: All.
brad requested review of this revision.
Herald added a project: Sanitizers.
Herald added a subscriber: Sanitizers.

Looks like when the VE support was added it was added a few lines below where it should have been.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D123439

Files:
  compiler-rt/lib/builtins/clear_cache.c


Index: compiler-rt/lib/builtins/clear_cache.c
===================================================================
--- compiler-rt/lib/builtins/clear_cache.c
+++ compiler-rt/lib/builtins/clear_cache.c
@@ -176,12 +176,12 @@
   arg.len = (uintptr_t)end - (uintptr_t)start;
 
   sysarch(RISCV_SYNC_ICACHE, &arg);
+#elif defined(__ve__)
+  __asm__ volatile("fencec 2");
 #else
 #if __APPLE__
   // On Darwin, sys_icache_invalidate() provides this functionality
   sys_icache_invalidate(start, end - start);
-#elif defined(__ve__)
-  __asm__ volatile("fencec 2");
 #else
   compilerrt_abort();
 #endif


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123439.421678.patch
Type: text/x-patch
Size: 596 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220409/82513b8d/attachment.bin>