SuccessChanges

Summary

  1. [libomptarget][nfc] Change unintentional target_impl prefix to kmpc_impl (details)
  2. [OpenMP][FIX] Generalize a test check line (details)
  3. Remove a redundant `default:` on an exhaustive switch(enum). (details)
  4. [CodeGen] Use CreateFNeg in buildFMulAdd (details)
  5. [CodeGen] Use IRBuilder::CreateFNeg for __builtin_conj (details)
Commit bc48af8c575abdd283d391194fa40d1419df5efe by jonathanchesterfield
[libomptarget][nfc] Change unintentional target_impl prefix to kmpc_impl
The file was modifiedopenmp/libomptarget/deviceRTLs/amdgcn/src/target_impl.h
The file was modifiedopenmp/libomptarget/deviceRTLs/common/src/libcall.cu
The file was modifiedopenmp/libomptarget/deviceRTLs/nvptx/src/target_impl.h
Commit 6bd1fcd795994f484e8f974be566edbbbf23927d by johannes
[OpenMP][FIX] Generalize a test check line
The new check line is compatible with the clang code generation check
line as it allows a 64 and 32 bit value.
I hope this makes the llvm-clang-win-x-armv7l buildbot happy.
The file was modifiedclang/test/OpenMP/parallel_codegen.cpp
Commit 07be32961a694670885626e72faca6f0007f14c9 by epastor
Remove a redundant `default:` on an exhaustive switch(enum).
The file was modifiedllvm/lib/Target/X86/X86AsmPrinter.cpp
Commit 8b23b2bbd9622c5f079a71c7078d167052f6a70c by craig.topper
[CodeGen] Use CreateFNeg in buildFMulAdd
We have an fneg instruction now and should use it instead of the fsub
-0.0 idiom. Looks like we had no test that showed that we handled the
negation cases here so I've added new tests.
Differential Revision: https://reviews.llvm.org/D72010
The file was modifiedclang/lib/CodeGen/CGExprScalar.cpp
The file was modifiedclang/test/CodeGen/fp-contract-pragma.cpp
Commit 70f8dd4cf604b2be3488895ef0d261154c1c1124 by craig.topper
[CodeGen] Use IRBuilder::CreateFNeg for __builtin_conj
This replaces the fsub -0.0 idiom with an fneg instruction. We didn't
see to have a test that showed the current codegen. Just some tests for
constant folding and a test that was only checking the declare lines for
libcalls. The latter just checked that we did not have a declare for
@conj when using __builtin_conj.
Differential Revision: https://reviews.llvm.org/D72012
The file was addedclang/test/CodeGen/complex-builtins-2.c
The file was modifiedclang/lib/CodeGen/CGBuiltin.cpp