OpenMP编译报错expected '#pragma omp' clause before '{' token及task用法咨询

OpenMP的#pragma指示为单行语法规则，将{直接拼接在#pragma omp parallel、#pragma omp single的同一行末尾时，编译器会将{识别为pragma的待解析子句，不符合语法要求，因此抛出expected ‘#pragma omp’ clause before ‘{’ token错误。
修正方式为将{移到#pragma行的下一行即可。
此外示例代码还存在两处语法类小问题：

• 变量名笔误：压栈操作的变量名写为processing_elec，应修正为processing_ele
• 同名变量重复定义：全局作用域和single块内重复定义了processing_ele，属于不规范写法

当前task的使用不符合业务逻辑要求，存在三处核心错误：

1. Task为异步执行逻辑，生成task后立刻读取global_res[processing_ele].need_to_push的判断完全无效。此时Test函数尚未被调度执行，读取到的是未更新的旧值，判断结果完全不符合预期。
2. 共享栈stack_ele未做并发保护。如果将压栈逻辑移到task内部，多线程同时操作栈会触发数据竞争，导致栈结构损坏、程序崩溃。
3. 缺少任务等待逻辑。当前代码在single块的循环跑完后会直接退出parallel域，此时可能还有未执行完的task被强制销毁，导致处理逻辑不完整。

如果你的业务逻辑是初始栈的所有元素处理完成后，无需二次处理压回栈的元素，可以参考如下修正代码：

#include <omp.h>

// 栈操作锁声明
omp_lock_t stack_lock;
omp_init_lock(&stack_lock);
omp_set_num_threads(2);

#pragma omp parallel
{
    #pragma omp single
    {
        while (!stack_ele.empty()) {
            int processing_ele = stack_edge_id.top();
            stack_ele.pop();
            
            // 显式指定变量属性，避免隐式规则导致的取值错误
            #pragma omp task firstprivate(processing_ele) shared(global_res, stack_ele, stack_lock)
            {
                Test(processing_ele);
                // 栈操作加锁避免并发竞争
                if (global_res[processing_ele].need_to_push) {
                    omp_set_lock(&stack_lock);
                    stack_ele.push(processing_ele);
                    omp_unset_lock(&stack_lock);
                }
            }
        }
        // 等待所有异步task执行完成再退出parallel域
        #pragma omp taskwait
    }
}

omp_destroy_lock(&stack_lock);

如果你的业务逻辑需要循环处理压回栈的元素，当前单线程弹栈的结构无法满足需求，需要调整为任务主动从共享栈取元素的生产者-消费者模型，避免single线程提前结束弹栈循环。 #

内容的提问来源于stack exchange，提问作者Joxixi