JonChesterfield added inline comments.
Comment at: clang/lib/CodeGen/CGOpenMPRuntimeAMDGCN.cpp:39
+ // return constant compile-time target-specific warp size
+ unsigned WarpSize = CGF.getTarget().getGridValue(llvm::omp::GV_Warp_Size);
+ return Bld.getInt32(WarpSize);
-
ABataev added inline comments.
Comment at: clang/lib/CodeGen/CGOpenMPRuntimeAMDGCN.cpp:39
+ // return constant compile-time target-specific warp size
+ unsigned WarpSize = CGF.getTarget().getGridValue(llvm::omp::GV_Warp_Size);
+ return Bld.getInt32(WarpSize);
JonChesterfield added a comment.
This is indeed the direction I had in mind. Broad strokes look right. I
probably wouldn't notice an accidental change amidst the whitespace reshuffle.
Very happy to read through line by line if you can split the whitespace change
out.
Comment
