[PATCH] D91054: [Clang][OpenMP] Frontend work for sections - D89671

[Chirag Khandelwal via Phabricator via cfe-commits]

AMDChirag updated this revision to Diff 337654.
AMDChirag added a comment.
Herald added a subscriber: jfb.

Fixed cancel construct and applied updates to its test cases

The test cases are updated so that the IR generated with and without IRBuilder are considered as they are different.

The cancel codegenerator with IRBuilder causes the IR to be created in the form:

  ...
  %29 = call i32 @__kmpc_cancel(%struct.ident_t* @1, i32 %omp_global_thread_num9, i32 3)
  %30 = icmp eq i32 %29, 0
  ...

Whereas, without IRBuilder, the IR created is in the form:

  ...
  %20 = call i32 @__kmpc_cancel(%struct.ident_t* @1, i32 %0, i32 3)
  %21 = icmp ne i32 %20, 0
  ...

In essence, the output is logically same, but semantically different, hence the requirement of updating test cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91054

Files:
  clang/lib/CodeGen/CGOpenMPRuntime.cpp
  clang/lib/CodeGen/CGStmtOpenMP.cpp
  clang/test/OpenMP/cancel_codegen.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91054.337654.patch
Type: text/x-patch
Size: 107017 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210415/298c40b4/attachment-0001.bin>