[clang] [flang] [mlir] [flang][flang-driver][mlir][OpenMP] atomic control support (PR #143441)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143441 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [mlir] [mlir][OpenMP][llvm]Atomic control backend (PR #143769)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143769 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143769 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/4] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143769 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143769 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/4] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143441 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/3] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143441 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/3] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -88,6 +88,46 @@ void fir::setTuneCPU(mlir::ModuleOp mod, llvm::StringRef cpu) { mod->setAttr(tuneCpuName, mlir::StringAttr::get(ctx, cpu)); } +static constexpr const char *amdgpuIgnoreDenormalModeName = +"fir.amdgpu.ignore.denormal.mode"; +void fir::setAmdgpuIgnoreDe

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -58,10 +58,24 @@ void setTargetCPU(mlir::ModuleOp mod, llvm::StringRef cpu); /// Get the target CPU string from the Module or return a null reference. llvm::StringRef getTargetCPU(mlir::ModuleOp mod); +// Setters and Getters for atomic control options anchu

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -53,6 +53,11 @@ class TargetOptions { /// Print verbose assembly bool asmVerbose = false; + + /// Atomic Control Options for AMD GPU anchuraj wrote: Updated https://github.com/llvm/llvm-project/pull/143441

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -58,10 +58,24 @@ void setTargetCPU(mlir::ModuleOp mod, llvm::StringRef cpu); /// Get the target CPU string from the Module or return a null reference. llvm::StringRef getTargetCPU(mlir::ModuleOp mod); +// Setters and Getters for atomic control options +void setAmdgpuIgnoreD

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -2911,8 +2911,13 @@ static void genAtomicUpdateStatement( if (rightHandClauseList) genOmpAtomicHintAndMemoryOrderClauses(converter, *rightHandClauseList, hint, memoryOrder); + auto module = firOpBuilder.getModule(); -

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

@@ -54,6 +54,20 @@ def FlagsAttr : OpenMP_Attr<"Flags", "flags"> { let assemblyFormat = "`<` struct(params) `>`"; } +//===--===// +// AtomicControlAttr +//===-

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

anchuraj wrote: > Thanks for this. > > Since you have added both `FlangOption` and `FC1Option` in `Options.td`, > could you add a test that checks that the option is handled by the main > driver as well as `fc1`. One that simply checks that the option gets passed > down to `fc1` from the main

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143769 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143769 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/4] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143441 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/3] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143769 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/4] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143441 >From c9b65080c1301c46b05fddc1278eaf2817963cb9 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/3] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143441 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [mlir] Atomic control backend (PR #143769)

https://github.com/anchuraj created https://github.com/llvm/llvm-project/pull/143769 None >From 51c6a6a9b1f58f37df3eb1b6988b62f489c8f4b8 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/2] [flang][flang-driver] atomic control support --- c

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/143441 >From 51c6a6a9b1f58f37df3eb1b6988b62f489c8f4b8 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Wed, 4 Jun 2025 15:12:49 -0500 Subject: [PATCH 1/2] [flang][flang-driver] atomic control support --- clang/i

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/143441 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [flang][flang-driver] atomic control support (PR #143441)

https://github.com/anchuraj created https://github.com/llvm/llvm-project/pull/143441 Atomic Control Options are used to specify architectural characteristics to help lowering of atomic operations. The options used are: `-f[no-]atomic-remote-memory`, `-f[no-]atomic-fine-grained-memory`, `-f[no

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

anchuraj wrote: > Hi @anchuraj! Nice patch! Are you interesting in extending the front-end > handling to support: > > ``` > -finstrument-functions-exclude-function-list=sym,sym,... > ``` > > ``` > -finstrument-functions-exclude-file-list=file,file,... > ``` > > In my experience, if found that

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj closed https://github.com/llvm/llvm-project/pull/137996 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

@@ -81,6 +81,8 @@ class CodeGenOptions : public CodeGenOptionsBase { /// Options to add to the linker for the object file std::vector DependentLibs; + bool InstrumentFunctions{false}; anchuraj wrote: Thank you for the review @tblah . Updated. https://gi

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/137996 >From bb486c5e7cbe7b1c4a87469e06ca51bf49ddd081 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Tue, 29 Apr 2025 14:41:55 -0500 Subject: [PATCH 1/4] [flang] Support flag -finstrument-functions --- clang/

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/137996 >From bb486c5e7cbe7b1c4a87469e06ca51bf49ddd081 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Tue, 29 Apr 2025 14:41:55 -0500 Subject: [PATCH 1/3] [flang] Support flag -finstrument-functions --- clang/

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

@@ -81,6 +81,8 @@ class CodeGenOptions : public CodeGenOptionsBase { /// Options to add to the linker for the object file std::vector DependentLibs; + bool InstrumentFunctions{false}; anchuraj wrote: Updated. Thank you! https://github.com/llvm/llvm-proj

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/137996 >From bb486c5e7cbe7b1c4a87469e06ca51bf49ddd081 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Tue, 29 Apr 2025 14:41:55 -0500 Subject: [PATCH 1/3] [flang] Support flag -finstrument-functions --- clang/

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj updated https://github.com/llvm/llvm-project/pull/137996 >From bb486c5e7cbe7b1c4a87469e06ca51bf49ddd081 Mon Sep 17 00:00:00 2001 From: Anchu Rajendran Date: Tue, 29 Apr 2025 14:41:55 -0500 Subject: [PATCH 1/2] [flang] Support flag -finstrument-functions --- clang/

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

@@ -310,6 +310,10 @@ static void parseCodeGenArgs(Fortran::frontend::CodeGenOptions &opts, args.filtered(clang::driver::options::OPT_fembed_offload_object_EQ)) opts.OffloadObjects.push_back(a->getValue()); + if (args.hasFlag(clang::driver::options::OPT_finstrument

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

@@ -124,6 +128,8 @@ struct MLIRToLLVMPassPipelineConfig : public FlangEPCallBacks { bool UnsafeFPMath = false; ///< Set unsafe-fp-math attribute for functions. bool NSWOnLoopVarInc = true; ///< Add nsw flag to loop variable increments. bool EnableOpenMP = false; ///< Ena

[clang] [flang] [flang][flang-driver] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj edited https://github.com/llvm/llvm-project/pull/137996 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang] Support flag -finstrument-functions (PR #137996)

https://github.com/anchuraj created https://github.com/llvm/llvm-project/pull/137996 `f-instrument-functions` helps in profiling functions. This PR adds support for the option by defining values for function attributes `instrument_function_entry` and `instrument_function_exit`. LLVM Backend a