[PATCH] D83361: [LLVM] Add libatomic load/store functions to TargetLibraryInfo

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 8 01:47:30 PDT 2020


rovka added a comment.

Don't you also have to set as Available/Unavailable when initializing the TLI?



================
Comment at: llvm/lib/Analysis/TargetLibraryInfo.cpp:1232
+  case LibFunc_atomic_load:
+  // void __atomic_load(size_t, void *, void *, int)
+  case LibFunc_atomic_store:
----------------
Nit: The comment should go above the case statement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83361





More information about the llvm-commits mailing list