[PATCH] D70726: [OpenMP50] Add parallel master construct

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: clang/test/OpenMP/parallel_master_codegen.cpp:144
+// CK4:       [[A_VAL:%.+]] = alloca i32
+// CK4:       [[A_CASTED:%.+]] = alloca i{{32|64}}
+// CK4:       [[ZERO:%.+]] = load i32, i32* [[A_VAL]]
----------------
I think it would be better to pass the triple in all the test runs. Some of your test runs do not include triple and are executed using the default (host-based) triple. Did you try to run the tests with the 32bit triples, for example?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70726