[PATCH] D129089: [BOLT] Fix concurrency bugs in the instrumentation runtime

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 10:09:48 PDT 2022


ayermolo added a comment.

In D129089#3628628 <https://reviews.llvm.org/D129089#3628628>, @yota9 wrote:

> @ayermolo The runtime libs such as instrumentation must not depend on external libraries like libstdc++. But in our (currently internal) implementation we are using __atomic_test_and_set, I think it is better option here which eliminates inline asm usage and is arch independent

Ah I see. I am all for eliminating inline assembly, but I leave this to @maksfb


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129089



More information about the llvm-commits mailing list