[PATCH] D152328: InstrProf - don't emit 64 bit atomic operations on 32 bit platforms

Sean Mollet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 18:06:08 PDT 2023


SeanMollet added a comment.

In D152328#4436381 <https://reviews.llvm.org/D152328#4436381>, @davidxl wrote:

> My understanding is that atomic update is enabled in some shared build environment thus causing problems for the 32 bit target. If that is not the case, it is perhaps better to use -fprofile-update option to control the behavior as suggested by Ellis.

It's enabled at any optimization level greater than O0 (I pasted the offending code in the previous reply). As far as I can tell, profile-update has no effect on it.


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

https://reviews.llvm.org/D152328



More information about the llvm-commits mailing list