[all-commits] [llvm/llvm-project] 8697bb: [clang] Use CPlusPlus language option instead of B...

Mariya Podchishchaeva via All-commits all-commits at lists.llvm.org
Thu Feb 8 05:32:08 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8697bbe2d4aed109520e83c6beab52196ec5b702
      https://github.com/llvm/llvm-project/commit/8697bbe2d4aed109520e83c6beab52196ec5b702
  Author: Mariya Podchishchaeva <mariya.podchishchaeva at intel.com>
  Date:   2024-02-08 (Thu, 08 Feb 2024)

  Changed paths:
    M clang/lib/Sema/SemaChecking.cpp
    M clang/test/SemaOpenCL/operators.cl

  Log Message:
  -----------
  [clang] Use CPlusPlus language option instead of Bool (#80975)

As it was pointed out in
https://github.com/llvm/llvm-project/pull/80724, we should not be
checking `getLangOpts().Bool` when determining something related to
logical operators, since it only indicates that bool keyword is present,
not which semantic logical operators have. As a side effect a missing
`-Wpointer-bool-conversion` in OpenCL C was restored since like C23,
OpenCL C has bool keyword but logical operators still return int.




More information about the All-commits mailing list