Skip to content

Commit

Permalink
Do not explicitly enable the LLVM atomic optimizer pass (#2523)
Browse files Browse the repository at this point in the history
See: https://reviews.llvm.org/D152649
The LLVM AMDGPU atomic optimizer pass is now enabled by default, and the
existing option to enable or disable it will be removed in favour of
setting the strategy to "None".
  • Loading branch information
jayfoad committed Jun 15, 2023
1 parent 0e2f6b0 commit f65d888
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions lgc/state/LgcContext.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,12 @@ void LgcContext::initialize() {
setOptionDefault("enable-phi-of-ops", "0");
setOptionDefault("simplifycfg-sink-common", "0");
setOptionDefault("amdgpu-vgpr-index-mode", "1"); // force VGPR indexing on GFX8
#if LLVM_MAIN_REVISION && LLVM_MAIN_REVISION < 464446
// Old version of the code
setOptionDefault("amdgpu-atomic-optimizations", "1");
#else
// New version of the code (also handles unknown version, which we treat as latest)
#endif
#if LLVM_MAIN_REVISION && LLVM_MAIN_REVISION < 463788
// Old version of the code
#else
Expand Down

0 comments on commit f65d888

Please sign in to comment.