[PATCH] D124724: [Clang][OpenMP] Add the support for floating-point variables for specific atomic clauses

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:5936-5939
+          CGF.Builder.CreateCast(X.getType()->hasSignedIntegerRepresentation()
+                                     ? llvm::Instruction::CastOps::SIToFP
+                                     : llvm::Instruction::CastOps::UIToFP,
+                                 IC, X.getAddress(CGF).getElementType());
----------------
Do we have a test for this? If `X` is not integer, I rather doubt that hasSignedIntegerRepresentation is meaningful here


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124724