[clang] 90bd723 - Add support of the next Ubuntu (Ubuntu 24.10 - Oracular Oriol)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2024-05-02T22:55:14+02:00
New Revision: 90bd7234e36e5b3d63155f481b739ea463985631

URL: 
https://github.com/llvm/llvm-project/commit/90bd7234e36e5b3d63155f481b739ea463985631
DIFF: 
https://github.com/llvm/llvm-project/commit/90bd7234e36e5b3d63155f481b739ea463985631.diff

LOG: Add support of the next Ubuntu (Ubuntu 24.10 - Oracular Oriol)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index a8de94163e8b0b..1404e168684821 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -79,6 +79,7 @@ class Distro {
 UbuntuLunar,
 UbuntuMantic,
 UbuntuNoble,
+UbuntuOracular,
 UnknownDistro
   };
 
@@ -130,7 +131,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuNoble;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuOracular;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index a7e7f169dc1419..6f49e641104ccd 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -95,6 +95,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("lunar", Distro::UbuntuLunar)
 .Case("mantic", Distro::UbuntuMantic)
 .Case("noble", Distro::UbuntuNoble)
+.Case("oracular", Distro::UbuntuOracular)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Improve support for expression messages in `static_assert` (PR #73234)

[Sylvestre Ledru via cfe-commits]

sylvestre wrote:

Merci à toi :)


https://github.com/llvm/llvm-project/pull/73234
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Improve support for expression messages in `static_assert` (PR #73234)

[Sylvestre Ledru via cfe-commits]

sylvestre wrote:

Seems it caused:
https://github.com/llvm/llvm-project/issues/73628


https://github.com/llvm/llvm-project/pull/73234
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6266b96 - Add support of the next Ubuntu (Ubuntu 24.04 - Noble Numbat)

[Sylvestre Ledru via cfe-commits]

Author: Graham Inggs
Date: 2023-11-01T21:01:19+01:00
New Revision: 6266b964202336a02f40007928719e060bc81694

URL: 
https://github.com/llvm/llvm-project/commit/6266b964202336a02f40007928719e060bc81694
DIFF: 
https://github.com/llvm/llvm-project/commit/6266b964202336a02f40007928719e060bc81694.diff

LOG: Add support of the next Ubuntu (Ubuntu 24.04 - Noble Numbat)

Co-authored-by: Sylvestre Ledru 

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 8291f6575a71146..a8de94163e8b0b7 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -78,6 +78,7 @@ class Distro {
 UbuntuKinetic,
 UbuntuLunar,
 UbuntuMantic,
+UbuntuNoble,
 UnknownDistro
   };
 
@@ -129,7 +130,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuMantic;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuNoble;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 005c31bd38893c6..36f828f8cae26d3 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -94,6 +94,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("kinetic", Distro::UbuntuKinetic)
 .Case("lunar", Distro::UbuntuLunar)
 .Case("mantic", Distro::UbuntuMantic)
+.Case("noble", Distro::UbuntuNoble)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 0517a9c - clang/Debian: add Debian Trixie now that it is in unstable

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2023-07-21T18:23:14+02:00
New Revision: 0517a9cae90fafbc36267a81df40ce4eef73eb30

URL: 
https://github.com/llvm/llvm-project/commit/0517a9cae90fafbc36267a81df40ce4eef73eb30
DIFF: 
https://github.com/llvm/llvm-project/commit/0517a9cae90fafbc36267a81df40ce4eef73eb30.diff

LOG: clang/Debian: add Debian Trixie now that it is in unstable

Added: 


Modified: 
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index e86b589a1cfb5e..6e0087565941ea 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -170,6 +170,7 @@ static Distro::DistroType 
DetectDistro(llvm::vfs::FileSystem ) {
 .Case("buster/sid", Distro::DebianBuster)
 .Case("bullseye/sid", Distro::DebianBullseye)
 .Case("bookworm/sid", Distro::DebianBookworm)
+.Case("trixie/sid", Distro::DebianTrixie)
 .Default(Distro::UnknownDistro);
   }
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 443f490 - Revert "[CUDA][HIP] Use the same default language std as C++"

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2023-07-20T10:54:54+02:00
New Revision: 443f490b6aac4af28500f4b09b68a13260f4fba9

URL: 
https://github.com/llvm/llvm-project/commit/443f490b6aac4af28500f4b09b68a13260f4fba9
DIFF: 
https://github.com/llvm/llvm-project/commit/443f490b6aac4af28500f4b09b68a13260f4fba9.diff

LOG: Revert "[CUDA][HIP] Use the same default language std as C++"

This reverts commit 2d1d07152bd26b001dedec3400b4b01d3bb11622.

Added: 


Modified: 
clang/docs/ReleaseNotes.rst
clang/include/clang/Basic/LangStandards.def
clang/lib/Basic/LangStandards.cpp
clang/test/CodeGenCUDA/long-double.cu
clang/test/Driver/unknown-std.cpp
clang/test/Preprocessor/lang-std.cpp

Removed: 




diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index ef1cc898c21f6d..2287ff9d70da3f 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -886,9 +886,6 @@ RISC-V Support
 
 CUDA/HIP Language Changes
 ^
-- Clang has been updated to align its default language standard for CUDA/HIP 
with
-  that of C++. The standard has now been enhanced to gnu++17, supplanting the
-  previously used c++14.
 
 CUDA Support
 

diff  --git a/clang/include/clang/Basic/LangStandards.def 
b/clang/include/clang/Basic/LangStandards.def
index 5c28bdd28ef257..911b626e4c9661 100644
--- a/clang/include/clang/Basic/LangStandards.def
+++ b/clang/include/clang/Basic/LangStandards.def
@@ -214,6 +214,14 @@ LANGSTANDARD_ALIAS_DEPR(openclcpp10, "CLC++")
 LANGSTANDARD_ALIAS_DEPR(openclcpp10, "CLC++1.0")
 LANGSTANDARD_ALIAS_DEPR(openclcpp2021, "CLC++2021")
 
+// CUDA
+LANGSTANDARD(cuda, "cuda", CUDA, "NVIDIA CUDA(tm)",
+ LineComment | CPlusPlus | CPlusPlus11 | CPlusPlus14 | Digraphs)
+
+// HIP
+LANGSTANDARD(hip, "hip", HIP, "HIP",
+ LineComment | CPlusPlus | CPlusPlus11 | CPlusPlus14 | Digraphs)
+
 // HLSL
 LANGSTANDARD(hlsl, "hlsl",
  HLSL, "High Level Shader Language",

diff  --git a/clang/lib/Basic/LangStandards.cpp 
b/clang/lib/Basic/LangStandards.cpp
index 033c0efe8d4f5d..c6a1a0acc2cb7e 100644
--- a/clang/lib/Basic/LangStandards.cpp
+++ b/clang/lib/Basic/LangStandards.cpp
@@ -54,6 +54,8 @@ LangStandard::Kind 
clang::getDefaultLanguageStandard(clang::Language Lang,
 return LangStandard::lang_opencl12;
   case Language::OpenCLCXX:
 return LangStandard::lang_openclcpp10;
+  case Language::CUDA:
+return LangStandard::lang_cuda;
   case Language::Asm:
   case Language::C:
 // The PS4 uses C99 as the default C standard.
@@ -64,13 +66,13 @@ LangStandard::Kind 
clang::getDefaultLanguageStandard(clang::Language Lang,
 return LangStandard::lang_gnu11;
   case Language::CXX:
   case Language::ObjCXX:
-  case Language::CUDA:
-  case Language::HIP:
 if (T.isPS())
   return LangStandard::lang_gnucxx14;
 return LangStandard::lang_gnucxx17;
   case Language::RenderScript:
 return LangStandard::lang_c99;
+  case Language::HIP:
+return LangStandard::lang_hip;
   case Language::HLSL:
 return LangStandard::lang_hlsl2021;
   }

diff  --git a/clang/test/CodeGenCUDA/long-double.cu 
b/clang/test/CodeGenCUDA/long-double.cu
index d52de972ea3da4..454a93ce5f6b6a 100644
--- a/clang/test/CodeGenCUDA/long-double.cu
+++ b/clang/test/CodeGenCUDA/long-double.cu
@@ -6,7 +6,7 @@
 // RUN:   -aux-triple x86_64-unknown-gnu-linux -fcuda-is-device \
 // RUN:   -emit-llvm -o - %s 2>&1 | FileCheck %s
 
-// CHECK: @_ZN15infinity_helperIeE5valueE = {{.*}} double 
0x47EFD586B834,{{.*}} align 8
+// CHECK: @_ZN15infinity_helperIeE5valueE = {{.*}} double 0x47EFD586B834, 
align 8
 // CHECK: @size = {{.*}} i32 8
 
 #include "Inputs/cuda.h"

diff  --git a/clang/test/Driver/unknown-std.cpp 
b/clang/test/Driver/unknown-std.cpp
index 5c58042a0a2c71..e918c087095ef0 100644
--- a/clang/test/Driver/unknown-std.cpp
+++ b/clang/test/Driver/unknown-std.cpp
@@ -4,10 +4,7 @@
 
 // RUN: not %clang %s -std=foobar -c 2>&1 | FileCheck --match-full-lines %s
 // RUN: not %clang -x objective-c++ %s -std=foobar -c 2>&1 | FileCheck 
--match-full-lines %s
-// RUN: not %clang -x cuda -nocudainc -nocudalib 
--cuda-path=%S/Inputs/CUDA/usr/local/cuda \
-// RUN:   %s -std=foobar -c 2>&1 | FileCheck --match-full-lines %s
-// RUN: not %clang -x hip -nocudainc -nocudalib %s -std=foobar -c 2>&1 \
-// RUN:   | FileCheck --match-full-lines %s
+// RUN: not %clang -x cuda -nocudainc -nocudalib 
--cuda-path=%S/Inputs/CUDA/usr/local/cuda %s -std=foobar -c 2>&1 | FileCheck 
--match-full-lines --check-prefix=CHECK --check-prefix=CUDA %s
 
 // CHECK: error: invalid value 'foobar' in '-std=foobar'
 // CHECK-NEXT: note: use 'c++98' or 'c++03' for 'ISO C++ 1998 with amendments' 
standard
@@ -24,6 +21,7 @@
 // CHECK-NEXT: note: use 'gnu++23' for 'ISO C++ 2023 DIS with GNU extensions' 
standard
 // CHECK-NEXT: note: use 'c++2c' or 'c++26' for 'Working draft for C++2c' 
standard
 // 

[clang] ddfb974 - Add support of the next Ubuntu (Ubuntu 23.10 - Mantic Minotaur)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2023-05-10T16:31:53+02:00
New Revision: ddfb974d0fca62e3eaeb98b79b5e29738c9082d2

URL: 
https://github.com/llvm/llvm-project/commit/ddfb974d0fca62e3eaeb98b79b5e29738c9082d2
DIFF: 
https://github.com/llvm/llvm-project/commit/ddfb974d0fca62e3eaeb98b79b5e29738c9082d2.diff

LOG: Add support of the next Ubuntu (Ubuntu 23.10 - Mantic Minotaur)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 0c858b3d716f1..8291f6575a711 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -77,6 +77,7 @@ class Distro {
 UbuntuJammy,
 UbuntuKinetic,
 UbuntuLunar,
+UbuntuMantic,
 UnknownDistro
   };
 
@@ -128,7 +129,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuLunar;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuMantic;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 85d82180a805c..e86b589a1cfb5 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -93,6 +93,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("jammy", Distro::UbuntuJammy)
 .Case("kinetic", Distro::UbuntuKinetic)
 .Case("lunar", Distro::UbuntuLunar)
+.Case("mantic", Distro::UbuntuMantic)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d1829c3 - Add support of the next Ubuntu (Ubuntu 23.04 - Lunar Lobster)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-11-05T22:35:54+01:00
New Revision: d1829c308da9c2adc46640a960c105b573db6555

URL: 
https://github.com/llvm/llvm-project/commit/d1829c308da9c2adc46640a960c105b573db6555
DIFF: 
https://github.com/llvm/llvm-project/commit/d1829c308da9c2adc46640a960c105b573db6555.diff

LOG: Add support of the next Ubuntu (Ubuntu 23.04 - Lunar Lobster)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 01d66b30b0386..1aaf93ddb7c43 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -76,6 +76,7 @@ class Distro {
 UbuntuImpish,
 UbuntuJammy,
 UbuntuKinetic,
+UbuntuLunar,
 UnknownDistro
   };
 
@@ -127,7 +128,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuKinetic;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuLunar;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 1898667279cc3..87a0c5a585115 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -92,6 +92,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("impish", Distro::UbuntuImpish)
 .Case("jammy", Distro::UbuntuJammy)
 .Case("kinetic", Distro::UbuntuKinetic)
+.Case("lunar", Distro::UbuntuLunar)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] b814d85 - Update links to googletest documentation

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-10-21T13:55:44+02:00
New Revision: b814d85e5b9d8acd146cef59141b2764107b3df9

URL: 
https://github.com/llvm/llvm-project/commit/b814d85e5b9d8acd146cef59141b2764107b3df9
DIFF: 
https://github.com/llvm/llvm-project/commit/b814d85e5b9d8acd146cef59141b2764107b3df9.diff

LOG: Update links to googletest documentation

Update links to googletest documentation
No automatic tests, but local manual test: i click it, it opens the googletest 
documentation.

Reviewed By: sylvestre.ledru

Differential Revision: https://reviews.llvm.org/D136424

Added: 


Modified: 
clang-tools-extra/clang-tidy/google/AvoidUnderscoreInGoogletestNameCheck.h

clang-tools-extra/docs/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.rst

Removed: 




diff  --git 
a/clang-tools-extra/clang-tidy/google/AvoidUnderscoreInGoogletestNameCheck.h 
b/clang-tools-extra/clang-tidy/google/AvoidUnderscoreInGoogletestNameCheck.h
index 53bfb737cb01a..b1f597a40a97c 100644
--- a/clang-tools-extra/clang-tidy/google/AvoidUnderscoreInGoogletestNameCheck.h
+++ b/clang-tools-extra/clang-tidy/google/AvoidUnderscoreInGoogletestNameCheck.h
@@ -17,7 +17,7 @@ namespace google {
 namespace readability {
 
 // Check for underscores in the names of googletest tests, per
-// 
https://github.com/google/googletest/blob/master/docs/faq.md#why-should-test-suite-names-and-test-names-not-contain-underscore
+// 
https://google.github.io/googletest/faq.html#why-should-test-suite-names-and-test-names-not-contain-underscore
 ///
 /// For the user-facing documentation see:
 /// 
http://clang.llvm.org/extra/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.html

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.rst
index 75b1a9ab5c73f..f2053b4d2fcd3 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/google/readability-avoid-underscore-in-googletest-name.rst
@@ -30,5 +30,5 @@ still checked.
 
 This check does not propose any fixes.
 
-.. _Underscores are not allowed: 
https://github.com/google/googletest/blob/master/googletest/docs/faq.md#why-should-test-suite-names-and-test-names-not-contain-underscore
-.. _disable individual tests: 
https://github.com/google/googletest/blob/master/googletest/docs/faq.md#why-should-test-suite-names-and-test-names-not-contain-underscore
+.. _Underscores are not allowed: 
https://google.github.io/googletest/faq.html#why-should-test-suite-names-and-test-names-not-contain-underscore
+.. _disable individual tests: 
https://google.github.io/googletest/advanced.html#temporarily-disabling-tests



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] cd20a18 - Revert "[clang, llvm] Add __declspec(safebuffers), support it in CodeView"

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-09-13T10:53:59+02:00
New Revision: cd20a1828605887699579789b5433111d5bc0319

URL: 
https://github.com/llvm/llvm-project/commit/cd20a1828605887699579789b5433111d5bc0319
DIFF: 
https://github.com/llvm/llvm-project/commit/cd20a1828605887699579789b5433111d5bc0319.diff

LOG: Revert "[clang, llvm] Add __declspec(safebuffers), support it in CodeView"

Causing:
https://github.com/llvm/llvm-project/issues/57709

This reverts commit ab56719acd98778fb2e48fa425ac7c8d27bdea86.

Added: 


Modified: 
clang/include/clang/Basic/Attr.td
clang/include/clang/Basic/AttrDocs.td
clang/lib/CodeGen/CodeGenModule.cpp
clang/test/Parser/MicrosoftExtensions.c
llvm/include/llvm/BinaryFormat/COFF.h
llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
llvm/lib/Target/AArch64/AArch64AsmPrinter.cpp
llvm/lib/Target/X86/X86AsmPrinter.cpp
llvm/test/DebugInfo/COFF/frameproc-flags.ll
llvm/test/DebugInfo/COFF/multifunction.ll
llvm/test/DebugInfo/COFF/simple.ll
llvm/test/DebugInfo/COFF/vframe-fpo.ll

Removed: 




diff  --git a/clang/include/clang/Basic/Attr.td 
b/clang/include/clang/Basic/Attr.td
index af0e08654014..0ad59b2c153a 100644
--- a/clang/include/clang/Basic/Attr.td
+++ b/clang/include/clang/Basic/Attr.td
@@ -2080,19 +2080,12 @@ def NotTailCalled : InheritableAttr {
 def : MutualExclusions<[AlwaysInline, NotTailCalled]>;
 
 def NoStackProtector : InheritableAttr {
-  let Spellings = [Clang<"no_stack_protector">, Declspec<"safebuffers">];
+  let Spellings = [Clang<"no_stack_protector">];
   let Subjects = SubjectList<[Function]>;
   let Documentation = [NoStackProtectorDocs];
   let SimpleHandler = 1;
 }
 
-def StrictGuardStackCheck : InheritableAttr {
-  let Spellings = [Declspec<"strict_gs_check">];
-  let Subjects = SubjectList<[Function]>;
-  let Documentation = [StrictGuardStackCheckDocs];
-  let SimpleHandler = 1;
-}
-
 def NoThrow : InheritableAttr {
   let Spellings = [GCC<"nothrow">, Declspec<"nothrow">];
   let Subjects = SubjectList<[FunctionLike]>;

diff  --git a/clang/include/clang/Basic/AttrDocs.td 
b/clang/include/clang/Basic/AttrDocs.td
index b6c15c92d581..68f1268ed7da 100644
--- a/clang/include/clang/Basic/AttrDocs.td
+++ b/clang/include/clang/Basic/AttrDocs.td
@@ -4453,8 +4453,7 @@ spelled "XYZ" in the `OpenMP 5.1 Standard`_).
 def NoStackProtectorDocs : Documentation {
   let Category = DocCatFunction;
   let Content = [{
-Clang supports the ``__attribute__((no_stack_protector))`` and Microsoft style
-``__declspec(safebuffers)`` attribute which disables
+Clang supports the ``__attribute__((no_stack_protector))`` attribute which 
disables
 the stack protector on the specified function. This attribute is useful for
 selectively disabling the stack protector on some functions when building with
 ``-fstack-protector`` compiler option.
@@ -4473,27 +4472,6 @@ option.
 }];
 }
 
-def StrictGuardStackCheckDocs : Documentation {
-  let Category = DocCatFunction;
-  let Content = [{
-Clang supports the Microsoft style ``__declspec((strict_gs_check))`` attribute
-which upgrades the stack protector check from ``-fstack-protector`` to
-``-fstack-protector-strong``.
-
-For example, it upgrades the stack protector for the function ``foo`` to
-``-fstack-protector-strong`` but function ``bar`` will still be built with the
-stack protector with the ``-fstack-protector`` option.
-
-.. code-block:: c
-
-__declspec((strict_gs_check))
-int foo(int x); // stack protection will be upgraded for foo.
-
-int bar(int y); // bar can be built with the standard stack protector 
checks.
-
-}];
-}
-
 def NotTailCalledDocs : Documentation {
   let Category = DocCatFunction;
   let Content = [{

diff  --git a/clang/lib/CodeGen/CodeGenModule.cpp 
b/clang/lib/CodeGen/CodeGenModule.cpp
index bbd42956a4b0..9edd618d359b 100644
--- a/clang/lib/CodeGen/CodeGenModule.cpp
+++ b/clang/lib/CodeGen/CodeGenModule.cpp
@@ -1970,17 +1970,14 @@ void 
CodeGenModule::SetLLVMFunctionAttributesForDefinition(const Decl *D,
   if (!hasUnwindExceptions(LangOpts))
 B.addAttribute(llvm::Attribute::NoUnwind);
 
-  if (D && D->hasAttr())
-; // Do nothing.
-  else if (D && D->hasAttr() &&
-   LangOpts.getStackProtector() == LangOptions::SSPOn)
-B.addAttribute(llvm::Attribute::StackProtectStrong);
-  else if (LangOpts.getStackProtector() == LangOptions::SSPOn)
-B.addAttribute(llvm::Attribute::StackProtect);
-  else if (LangOpts.getStackProtector() == LangOptions::SSPStrong)
-B.addAttribute(llvm::Attribute::StackProtectStrong);
-  else if (LangOpts.getStackProtector() == LangOptions::SSPReq)
-B.addAttribute(llvm::Attribute::StackProtectReq);
+  if (!D || !D->hasAttr()) {
+if (LangOpts.getStackProtector() == LangOptions::SSPOn)
+  B.addAttribute(llvm::Attribute::StackProtect);
+else if (LangOpts.getStackProtector() == LangOptions::SSPStrong)
+  

[clang] 8f555a5 - [cmake] Fix tablegen exports

[Sylvestre Ledru via cfe-commits]

Author: Nikita Popov
Date: 2022-08-16T14:17:23+02:00
New Revision: 8f555a52e033ceec4c4508eb800c9a186acec87f

URL: 
https://github.com/llvm/llvm-project/commit/8f555a52e033ceec4c4508eb800c9a186acec87f
DIFF: 
https://github.com/llvm/llvm-project/commit/8f555a52e033ceec4c4508eb800c9a186acec87f.diff

LOG: [cmake] Fix tablegen exports

This fixes some fallout from D131282. Currently, add_tablegen() will add the 
tablegen target to LLVM_EXPORTS and associates the install with LLVMExports. 
For non-standalone builds, this means that you end up with mlir-tblgen and 
clang-tblgen in LLVMExports.

However, these projects should instead be using MLIR_EXPORTS/MLIRTargets and 
CLANG_EXPORTS/ClangTargets. To fix this, add an extra EXPORT option and make 
use of get_target_export_arg() to create the correct export argument.

Reviewed By: ashay-github

Differential Revision: https://reviews.llvm.org/D131565

Added: 


Modified: 
clang/utils/TableGen/CMakeLists.txt
llvm/cmake/modules/TableGen.cmake
llvm/utils/TableGen/CMakeLists.txt
mlir/tools/mlir-pdll/CMakeLists.txt
mlir/tools/mlir-tblgen/CMakeLists.txt

Removed: 




diff  --git a/clang/utils/TableGen/CMakeLists.txt 
b/clang/utils/TableGen/CMakeLists.txt
index d2cb29dd23a30..4666d4d7aa5ce 100644
--- a/clang/utils/TableGen/CMakeLists.txt
+++ b/clang/utils/TableGen/CMakeLists.txt
@@ -1,6 +1,8 @@
 set(LLVM_LINK_COMPONENTS Support)
 
-add_tablegen(clang-tblgen CLANG DESTINATION "${CLANG_TOOLS_INSTALL_DIR}"
+add_tablegen(clang-tblgen CLANG
+  DESTINATION "${CLANG_TOOLS_INSTALL_DIR}"
+  EXPORT Clang
   ASTTableGen.cpp
   ClangASTNodesEmitter.cpp
   ClangASTPropertiesEmitter.cpp

diff  --git a/llvm/cmake/modules/TableGen.cmake 
b/llvm/cmake/modules/TableGen.cmake
index 172621d377485..59f6a1466078f 100644
--- a/llvm/cmake/modules/TableGen.cmake
+++ b/llvm/cmake/modules/TableGen.cmake
@@ -2,6 +2,7 @@
 # while LLVM_TARGET_DEPENDS may contain additional file dependencies.
 # Extra parameters for `tblgen' may come after `ofn' parameter.
 # Adds the name of the generated file to TABLEGEN_OUTPUT.
+include(LLVMDistributionSupport)
 
 function(tablegen project ofn)
   cmake_parse_arguments(ARG "" "" "DEPENDS;EXTRA_INCLUDES" ${ARGN})
@@ -140,7 +141,7 @@ function(add_public_tablegen_target target)
 endfunction()
 
 macro(add_tablegen target project)
-  cmake_parse_arguments(ADD_TABLEGEN "" "DESTINATION" "" ${ARGN})
+  cmake_parse_arguments(ADD_TABLEGEN "" "DESTINATION;EXPORT" "" ${ARGN})
 
   set(${target}_OLD_LLVM_LINK_COMPONENTS ${LLVM_LINK_COMPONENTS})
   set(LLVM_LINK_COMPONENTS ${LLVM_LINK_COMPONENTS} TableGen)
@@ -190,14 +191,12 @@ macro(add_tablegen target project)
   endif()
 
   if (ADD_TABLEGEN_DESTINATION AND NOT LLVM_INSTALL_TOOLCHAIN_ONLY AND 
LLVM_BUILD_UTILS)
-set(export_to_llvmexports)
-if(${target} IN_LIST LLVM_DISTRIBUTION_COMPONENTS OR
-NOT LLVM_DISTRIBUTION_COMPONENTS)
-  set(export_to_llvmexports EXPORT LLVMExports)
+set(export_arg)
+if(ADD_TABLEGEN_EXPORT)
+  get_target_export_arg(${target} ${ADD_TABLEGEN_EXPORT} export_arg)
 endif()
-
 install(TARGETS ${target}
-${export_to_llvmexports}
+${export_arg}
 COMPONENT ${target}
 RUNTIME DESTINATION "${ADD_TABLEGEN_DESTINATION}")
 if(NOT LLVM_ENABLE_IDE)
@@ -206,5 +205,8 @@ macro(add_tablegen target project)
COMPONENT ${target})
 endif()
   endif()
-  set_property(GLOBAL APPEND PROPERTY LLVM_EXPORTS ${target})
+  if(ADD_TABLEGEN_EXPORT)
+string(TOUPPER ${ADD_TABLEGEN_EXPORT} export_upper)
+set_property(GLOBAL APPEND PROPERTY ${export_upper}_EXPORTS ${target})
+  endif()
 endmacro()

diff  --git a/llvm/utils/TableGen/CMakeLists.txt 
b/llvm/utils/TableGen/CMakeLists.txt
index bf4d51b8dca44..56035bd0e4e0b 100644
--- a/llvm/utils/TableGen/CMakeLists.txt
+++ b/llvm/utils/TableGen/CMakeLists.txt
@@ -2,7 +2,9 @@ add_subdirectory(GlobalISel)
 
 set(LLVM_LINK_COMPONENTS Support)
 
-add_tablegen(llvm-tblgen LLVM DESTINATION "${LLVM_TOOLS_INSTALL_DIR}"
+add_tablegen(llvm-tblgen LLVM
+  DESTINATION "${LLVM_TOOLS_INSTALL_DIR}"
+  EXPORT LLVM
   AsmMatcherEmitter.cpp
   AsmWriterEmitter.cpp
   AsmWriterInst.cpp

diff  --git a/mlir/tools/mlir-pdll/CMakeLists.txt 
b/mlir/tools/mlir-pdll/CMakeLists.txt
index 6acee39e875cd..67b65d7ad5723 100644
--- a/mlir/tools/mlir-pdll/CMakeLists.txt
+++ b/mlir/tools/mlir-pdll/CMakeLists.txt
@@ -12,7 +12,9 @@ set(LIBS
   MLIRPDLLParser
   )
 
-add_tablegen(mlir-pdll MLIR_PDLL DESTINATION "${MLIR_TOOLS_INSTALL_DIR}"
+add_tablegen(mlir-pdll MLIR_PDLL
+  DESTINATION "${MLIR_TOOLS_INSTALL_DIR}"
+  EXPORT MLIR
   mlir-pdll.cpp
 
   DEPENDS

diff  --git a/mlir/tools/mlir-tblgen/CMakeLists.txt 
b/mlir/tools/mlir-tblgen/CMakeLists.txt
index 85f9207f723b1..afc3ab1853986 100644
--- a/mlir/tools/mlir-tblgen/CMakeLists.txt
+++ b/mlir/tools/mlir-tblgen/CMakeLists.txt
@@ -4,7 

[clang-tools-extra] 556c422 - clang-tidy doc: fix some typos

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-08-01T18:57:36+02:00
New Revision: 556c422de1c3b0623e83c85b36a43dd743660fb1

URL: 
https://github.com/llvm/llvm-project/commit/556c422de1c3b0623e83c85b36a43dd743660fb1
DIFF: 
https://github.com/llvm/llvm-project/commit/556c422de1c3b0623e83c85b36a43dd743660fb1.diff

LOG: clang-tidy doc: fix some typos

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/bugprone/signed-char-misuse.rst

clang-tools-extra/docs/clang-tidy/checks/bugprone/unchecked-optional-access.rst
clang-tools-extra/docs/clang-tidy/checks/cert/err33-c.rst

clang-tools-extra/docs/clang-tidy/checks/misc/throw-by-value-catch-by-reference.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/bugprone/signed-char-misuse.rst 
b/clang-tools-extra/docs/clang-tidy/checks/bugprone/signed-char-misuse.rst
index d0b18bd4595c7..72860e8cf2a1d 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/bugprone/signed-char-misuse.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/bugprone/signed-char-misuse.rst
@@ -47,7 +47,7 @@ A good example from the CERT description when a ``char`` 
variable is used to
 read from a file that might contain non-ASCII characters. The problem comes
 up when the code uses the ``-1`` integer value as EOF, while the 255 character
 code is also stored as ``-1`` in two's complement form of char type.
-See a simple example of this bellow. This code stops not only when it reaches
+See a simple example of this below. This code stops not only when it reaches
 the end of the file, but also when it gets a character with the 255 code.
 
 .. code-block:: c++

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/bugprone/unchecked-optional-access.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/bugprone/unchecked-optional-access.rst
index 3ed6372840a89..6e9a4e02e6d0e 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/bugprone/unchecked-optional-access.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/bugprone/unchecked-optional-access.rst
@@ -269,7 +269,7 @@ The check is aware of aliases of optional types that are 
created via
 Lambdas
 ---
 
-The check does not currently report unsafe optional acceses in lambdas.
+The check does not currently report unsafe optional accesses in lambdas.
 A future version will expand the scope to lambdas, following the rules
 outlined above. It is best to follow the same principles when using
 optionals in lambdas.

diff  --git a/clang-tools-extra/docs/clang-tidy/checks/cert/err33-c.rst 
b/clang-tools-extra/docs/clang-tidy/checks/cert/err33-c.rst
index 4a05629d6070c..c75ee8d3c09d7 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/cert/err33-c.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/cert/err33-c.rst
@@ -5,7 +5,7 @@ cert-err33-c
 
 Warns on unused function return values. Many of the standard library functions
 return a value that indicates if the call was successful. Ignoring the returned
-value can cause unexpected behavior if an error has occured. The following
+value can cause unexpected behavior if an error has occurred. The following
 functions are checked:
 
 * aligned_alloc()

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/misc/throw-by-value-catch-by-reference.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/misc/throw-by-value-catch-by-reference.rst
index c693387d9e4bf..af6ec1416e5e2 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/misc/throw-by-value-catch-by-reference.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/misc/throw-by-value-catch-by-reference.rst
@@ -17,7 +17,7 @@ Exceptions:
 are not susceptible to slicing and the usage of string literals is
 idiomatic.
   * Catching character pointers (``char``, ``wchar_t``, unicode character 
types)
-will not be flagged to allow catching sting literals.
+will not be flagged to allow catching string literals.
   * Moved named values will not be flagged as not throwing an anonymous
 temporary. In this case we can be sure that the user knows that the object
 can't be accessed outside catch blocks handling the error.



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 37047a2 - misc-const-correctness: fix the link to readability-isolate-declaration

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-08-01T18:52:53+02:00
New Revision: 37047a26731a6731c949682e159fe40786e6d711

URL: 
https://github.com/llvm/llvm-project/commit/37047a26731a6731c949682e159fe40786e6d711
DIFF: 
https://github.com/llvm/llvm-project/commit/37047a26731a6731c949682e159fe40786e6d711.diff

LOG: misc-const-correctness: fix the link to readability-isolate-declaration

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/misc/const-correctness.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/misc/const-correctness.rst 
b/clang-tools-extra/docs/clang-tidy/checks/misc/const-correctness.rst
index 852d9346576da..76d283da37dd1 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/misc/const-correctness.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/misc/const-correctness.rst
@@ -41,7 +41,7 @@ The check can analyzes values, pointers and references but 
not (yet) pointees:
 
 The automatic code transformation is only applied to variables that are 
declared in single
 declarations. You may want to prepare your code base with
-`readability-isolate-declaration `_ 
first.
+`readability-isolate-declaration <../readability/isolate-declaration.html>`_ 
first.
 
 Note that there is the check
 `cppcoreguidelines-avoid-non-const-global-variables 
`_



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6f4dc5d - Add support of the next Ubuntu (Ubuntu 22.10 - Kinetic Kudu)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-05-23T13:49:34+02:00
New Revision: 6f4dc5dae60598e71c8211b0c1a705474b0a223f

URL: 
https://github.com/llvm/llvm-project/commit/6f4dc5dae60598e71c8211b0c1a705474b0a223f
DIFF: 
https://github.com/llvm/llvm-project/commit/6f4dc5dae60598e71c8211b0c1a705474b0a223f.diff

LOG: Add support of the next Ubuntu (Ubuntu 22.10 - Kinetic Kudu)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index c31fbd4e93f1..01d66b30b038 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -75,6 +75,7 @@ class Distro {
 UbuntuHirsute,
 UbuntuImpish,
 UbuntuJammy,
+UbuntuKinetic,
 UnknownDistro
   };
 
@@ -126,7 +127,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuJammy;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuKinetic;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index b0e2f3ac94c7..1898667279cc 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -91,6 +91,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("hirsute", Distro::UbuntuHirsute)
 .Case("impish", Distro::UbuntuImpish)
 .Case("jammy", Distro::UbuntuJammy)
+.Case("kinetic", Distro::UbuntuKinetic)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] a020006 - Add support of the next Debian (Debian 13 - Trixie)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-05-23T13:47:13+02:00
New Revision: a02000611a8fc4843f7a3077d8bb4840ef08de8b

URL: 
https://github.com/llvm/llvm-project/commit/a02000611a8fc4843f7a3077d8bb4840ef08de8b
DIFF: 
https://github.com/llvm/llvm-project/commit/a02000611a8fc4843f7a3077d8bb4840ef08de8b.diff

LOG: Add support of the next Debian (Debian 13 - Trixie)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 2723f75e89458..c31fbd4e93f1a 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -38,6 +38,7 @@ class Distro {
 DebianBuster,
 DebianBullseye,
 DebianBookworm,
+DebianTrixie,
 Exherbo,
 RHEL5,
 RHEL6,
@@ -121,7 +122,7 @@ class Distro {
   bool IsOpenSUSE() const { return DistroVal == OpenSUSE; }
 
   bool IsDebian() const {
-return DistroVal >= DebianLenny && DistroVal <= DebianBookworm;
+return DistroVal >= DebianLenny && DistroVal <= DebianTrixie;
   }
 
   bool IsUbuntu() const {

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 5ac38c34d1128..b0e2f3ac94c75 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -153,6 +153,8 @@ static Distro::DistroType 
DetectDistro(llvm::vfs::FileSystem ) {
 return Distro::DebianBullseye;
   case 12:
 return Distro::DebianBookworm;
+  case 13:
+return Distro::DebianTrixie;
   default:
 return Distro::UnknownDistro;
   }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] f2c2e92 - Fix a typo (occured => occurred)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-02-08T21:35:26+01:00
New Revision: f2c2e924e73a76562b7cc6d8fd96df5b177eaf18

URL: 
https://github.com/llvm/llvm-project/commit/f2c2e924e73a76562b7cc6d8fd96df5b177eaf18
DIFF: 
https://github.com/llvm/llvm-project/commit/f2c2e924e73a76562b7cc6d8fd96df5b177eaf18.diff

LOG: Fix a typo (occured => occurred)

Reported:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1005195

Added: 


Modified: 
clang/include/clang-c/Index.h
clang/include/clang/Analysis/Analyses/CalledOnceCheck.h
clang/include/clang/Basic/AttrDocs.td
clang/include/clang/Basic/Specifiers.h
clang/include/clang/CrossTU/CrossTranslationUnit.h
clang/include/clang/Format/Format.h
clang/include/clang/Sema/Sema.h
clang/lib/Sema/SemaOpenMP.cpp
clang/lib/StaticAnalyzer/Core/RegionStore.cpp
clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp
clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp
llvm/include/llvm/ADT/IntrusiveRefCntPtr.h
llvm/include/llvm/CodeGen/ISDOpcodes.h
llvm/include/llvm/Support/MathExtras.h
llvm/lib/DebugInfo/DWARF/DWARFUnit.cpp
llvm/lib/TableGen/TGLexer.h
llvm/lib/Transforms/IPO/Attributor.cpp
llvm/test/Transforms/LowerConstantIntrinsics/stale-worklist-phi.ll
llvm/utils/reduce_pipeline_test/fake_opt.py

Removed: 




diff  --git a/clang/include/clang-c/Index.h b/clang/include/clang-c/Index.h
index b0d7ef509c26f..ea44673a3f1ea 100644
--- a/clang/include/clang-c/Index.h
+++ b/clang/include/clang-c/Index.h
@@ -3900,7 +3900,7 @@ enum CXTypeNullabilityKind {
   /**
* Generally behaves like Nullable, except when used in a block parameter 
that
* was imported into a swift async method. There, swift will assume that the
-   * parameter can get null even if no error occured. _Nullable parameters are
+   * parameter can get null even if no error occurred. _Nullable parameters are
* assumed to only get null on error.
*/
   CXTypeNullability_NullableResult = 4

diff  --git a/clang/include/clang/Analysis/Analyses/CalledOnceCheck.h 
b/clang/include/clang/Analysis/Analyses/CalledOnceCheck.h
index 6d78fa4a897ad..b65a3967cd50d 100644
--- a/clang/include/clang/Analysis/Analyses/CalledOnceCheck.h
+++ b/clang/include/clang/Analysis/Analyses/CalledOnceCheck.h
@@ -79,7 +79,7 @@ class CalledOnceCheckHandler {
   /// the path containing the call and not containing the call.  This helps us
   /// to pinpoint a bad path for the user.
   /// \param Parameter -- parameter that should be called once.
-  /// \param Function -- function declaration where the problem occured.
+  /// \param Function -- function declaration where the problem occurred.
   /// \param Where -- the least common ancestor statement.
   /// \param Reason -- a reason describing the path without a call.
   /// \param IsCalledDirectly -- true, if parameter actually gets called on

diff  --git a/clang/include/clang/Basic/AttrDocs.td 
b/clang/include/clang/Basic/AttrDocs.td
index 1682a71eb992e..74ee5f84dfa85 100644
--- a/clang/include/clang/Basic/AttrDocs.td
+++ b/clang/include/clang/Basic/AttrDocs.td
@@ -3816,10 +3816,10 @@ completion handler in a Swift async method. For 
instance, here:
 This method asynchronously calls ``completionHandler`` when the data is
 available, or calls it with an error. ``_Nullable_result`` indicates to the
 Swift importer that this is the uncommon case where ``result`` can get ``nil``
-even if no error has occured, and will therefore import it as a Swift optional
+even if no error has occurred, and will therefore import it as a Swift optional
 type. Otherwise, if ``result`` was annotated with ``_Nullable``, the Swift
 importer will assume that ``result`` will always be non-nil unless an error
-occured.
+occurred.
 }];
 }
 

diff  --git a/clang/include/clang/Basic/Specifiers.h 
b/clang/include/clang/Basic/Specifiers.h
index 82b3c1f3c2740..52ca7cce9d961 100644
--- a/clang/include/clang/Basic/Specifiers.h
+++ b/clang/include/clang/Basic/Specifiers.h
@@ -324,7 +324,7 @@ namespace clang {
 Unspecified,
 // Generally behaves like Nullable, except when used in a block parameter
 // that was imported into a swift async method. There, swift will assume
-// that the parameter can get null even if no error occured. _Nullable
+// that the parameter can get null even if no error occurred. _Nullable
 // parameters are assumed to only get null on error.
 NullableResult,
   };

diff  --git a/clang/include/clang/CrossTU/CrossTranslationUnit.h 
b/clang/include/clang/CrossTU/CrossTranslationUnit.h
index d9f9c51fccd9c..a09826f93afc0 100644
--- a/clang/include/clang/CrossTU/CrossTranslationUnit.h
+++ b/clang/include/clang/CrossTU/CrossTranslationUnit.h
@@ -291,7 +291,7 @@ class CrossTranslationUnitContext {
 /// \param DisplayCTUProgress Display a message about loading new ASTs.
 ///
 /// \return An Expected instance which 

[clang] b4bb622 - Also document -arch as -arch is mac specific

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2022-02-02T16:23:23+01:00
New Revision: b4bb622fc94a051dd9452930b5f79af688fc101b

URL: 
https://github.com/llvm/llvm-project/commit/b4bb622fc94a051dd9452930b5f79af688fc101b
DIFF: 
https://github.com/llvm/llvm-project/commit/b4bb622fc94a051dd9452930b5f79af688fc101b.diff

LOG: Also document -arch as -arch is mac specific

Reviewed By: serge-sans-paille

Differential Revision: https://reviews.llvm.org/D60380

Added: 


Modified: 
clang/docs/CommandGuide/clang.rst

Removed: 




diff  --git a/clang/docs/CommandGuide/clang.rst 
b/clang/docs/CommandGuide/clang.rst
index 6797020d1b68e..aec62789a43e6 100644
--- a/clang/docs/CommandGuide/clang.rst
+++ b/clang/docs/CommandGuide/clang.rst
@@ -323,7 +323,11 @@ number of cross compilers, or may only support a native 
target.
 
 .. option:: -arch 
 
-  Specify the architecture to build for.
+  Specify the architecture to build for (Mac OS X specific).
+
+.. option:: -target 
+
+  Specify the architecture to build for (all platforms).
 
 .. option:: -mmacosx-version-min=
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fd71493 - Add missing namespace to PPCLinux.cpp

[Sylvestre Ledru via cfe-commits]

Author: Mike Hommey
Date: 2022-01-27T09:26:46+01:00
New Revision: fd71493ff0602d7d81763263df2f2074fe0db094

URL: 
https://github.com/llvm/llvm-project/commit/fd71493ff0602d7d81763263df2f2074fe0db094
DIFF: 
https://github.com/llvm/llvm-project/commit/fd71493ff0602d7d81763263df2f2074fe0db094.diff

LOG: Add missing namespace to PPCLinux.cpp

This fixes a build failure with MSVC introduced in
https://reviews.llvm.org/D112906

Reviewed By: nemanjai

Differential Revision: https://reviews.llvm.org/D118211

Added: 


Modified: 
clang/lib/Driver/ToolChains/PPCLinux.cpp

Removed: 




diff  --git a/clang/lib/Driver/ToolChains/PPCLinux.cpp 
b/clang/lib/Driver/ToolChains/PPCLinux.cpp
index e5e1aa06f4b1d..e480d8bd8703c 100644
--- a/clang/lib/Driver/ToolChains/PPCLinux.cpp
+++ b/clang/lib/Driver/ToolChains/PPCLinux.cpp
@@ -13,6 +13,7 @@
 #include "llvm/Support/FileSystem.h"
 #include "llvm/Support/Path.h"
 
+using namespace clang::driver;
 using namespace clang::driver::toolchains;
 using namespace llvm::opt;
 using namespace llvm::sys;



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] d96f92f - fix typos in comments

[Sylvestre Ledru via cfe-commits]

Author: Zhuo Zhang
Date: 2021-11-29T14:06:33+01:00
New Revision: d96f92ff16edab72cf78811673f02371f07a5a70

URL: 
https://github.com/llvm/llvm-project/commit/d96f92ff16edab72cf78811673f02371f07a5a70
DIFF: 
https://github.com/llvm/llvm-project/commit/d96f92ff16edab72cf78811673f02371f07a5a70.diff

LOG: fix typos in comments

Added: 


Modified: 
clang-tools-extra/clang-doc/ClangDoc.h
llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp
llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections.yaml

Removed: 




diff  --git a/clang-tools-extra/clang-doc/ClangDoc.h 
b/clang-tools-extra/clang-doc/ClangDoc.h
index f3820d2c71d27..1f0c43d70db17 100644
--- a/clang-tools-extra/clang-doc/ClangDoc.h
+++ b/clang-tools-extra/clang-doc/ClangDoc.h
@@ -6,7 +6,7 @@
 //
 
//===--===//
 //
-// This file exposes a method to craete the FrontendActionFactory for the
+// This file exposes a method to create the FrontendActionFactory for the
 // clang-doc tool. The factory runs the clang-doc mapper on a given set of
 // source code files, storing the results key-value pairs in its
 // ExecutionContext.

diff  --git a/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp 
b/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp
index 1b7fdb5882756..0de76ab78e0fd 100644
--- a/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp
+++ b/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp
@@ -1301,7 +1301,7 @@ RuntimeDyldELF::processRelocationRef(
 MemMgr.allowStubAllocation()) {
   resolveAArch64Branch(SectionID, Value, RelI, Stubs);
 } else if (RelType == ELF::R_AARCH64_ADR_GOT_PAGE) {
-  // Craete new GOT entry or find existing one. If GOT entry is
+  // Create new GOT entry or find existing one. If GOT entry is
   // to be created, then we also emit ABS64 relocation for it.
   uint64_t GOTOffset = findOrAllocGOTEntry(Value, ELF::R_AARCH64_ABS64);
   resolveGOTOffsetRelocation(SectionID, Offset, GOTOffset + Addend,

diff  --git a/llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections.yaml 
b/llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections.yaml
index 9a71b3175efaa..8a534332a6525 100644
--- a/llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections.yaml
+++ b/llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections.yaml
@@ -33,7 +33,7 @@ Sections:
   - Name:.data3
 # The last section not only overlaps segment boundary, but
 # also has linear address which doesn't fit 20 bits. The 
-# following records should be craeted:
+# following records should be created:
 # 'SegmentAddr'(02) record with address 0x0
 # 'ExtendedAddr'(04) record with address 0x10
 # 'Data'(00) record with 8 bytes of section data



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] a709787 - Add support of the next Ubuntu (Ubuntu 22.04 - Jammy Jellyfish)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-10-23T23:55:50+02:00
New Revision: a709787cd988aaca847995bd08cc9348c9c6c956

URL: 
https://github.com/llvm/llvm-project/commit/a709787cd988aaca847995bd08cc9348c9c6c956
DIFF: 
https://github.com/llvm/llvm-project/commit/a709787cd988aaca847995bd08cc9348c9c6c956.diff

LOG: Add support of the next Ubuntu (Ubuntu 22.04 - Jammy Jellyfish)

It is going to be a LTS release

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index d9909bcf96968..2723f75e89458 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -73,6 +73,7 @@ class Distro {
 UbuntuGroovy,
 UbuntuHirsute,
 UbuntuImpish,
+UbuntuJammy,
 UnknownDistro
   };
 
@@ -124,7 +125,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuImpish;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuJammy;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 6a11bacb41e5b..5ac38c34d1128 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -90,6 +90,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("groovy", Distro::UbuntuGroovy)
 .Case("hirsute", Distro::UbuntuHirsute)
 .Case("impish", Distro::UbuntuImpish)
+.Case("jammy", Distro::UbuntuJammy)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fd5e3f3 - Replace references to Makefile.sphinx

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-10-22T15:32:12+02:00
New Revision: fd5e3f36f2186855966978c0e11b716277eee387

URL: 
https://github.com/llvm/llvm-project/commit/fd5e3f36f2186855966978c0e11b716277eee387
DIFF: 
https://github.com/llvm/llvm-project/commit/fd5e3f36f2186855966978c0e11b716277eee387.diff

LOG: Replace references to Makefile.sphinx

and fix some typos

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D112299

Added: 


Modified: 
clang/include/clang/Basic/AttrDocs.td
llvm/docs/README.txt

Removed: 




diff  --git a/clang/include/clang/Basic/AttrDocs.td 
b/clang/include/clang/Basic/AttrDocs.td
index 71f947687a08f..09477d6c4423d 100644
--- a/clang/include/clang/Basic/AttrDocs.td
+++ b/clang/include/clang/Basic/AttrDocs.td
@@ -22,7 +22,7 @@
 // Windows (from within the clang\docs directory):
 //   make.bat html
 // Non-Windows (from within the clang\docs directory):
-//   make -f Makefile.sphinx html
+//   sphinx-build -b html _build/html
 
 def GlobalDocumentation {
   code Intro =[{..

diff  --git a/llvm/docs/README.txt b/llvm/docs/README.txt
index c36a533fb0050..074606c7bd36e 100644
--- a/llvm/docs/README.txt
+++ b/llvm/docs/README.txt
@@ -14,7 +14,7 @@ Sphinx  and then do:
 cd 
 cmake -DLLVM_ENABLE_SPHINX=true -DSPHINX_OUTPUT_HTML=true 
 make -j3 docs-llvm-html
-$BROWSER /docs//html/index.html
+$BROWSER /docs/html/index.html
 
 The mapping between reStructuredText files and generated documentation is
 `docs/Foo.rst` <-> `/docs//html/Foo.html` <-> 
`https://llvm.org/docs/Foo.html`.
@@ -35,7 +35,7 @@ directory `/docs/man/`.
 cd 
 cmake -DLLVM_ENABLE_SPHINX=true -DSPHINX_OUTPUT_MAN=true 
 make -j3 docs-llvm-man
-man -l >build-dir>/docs/man/FileCheck.1
+man -l /docs/man/FileCheck.1
 
 The correspondence between .rst files and man pages is
 `docs/CommandGuide/Foo.rst` <-> `/docs//man/Foo.1`.
@@ -49,8 +49,9 @@ Checking links
 The reachability of external links in the documentation can be checked by
 running:
 
-cd docs/
-make -f Makefile.sphinx linkcheck
+cd llvm/docs/
+sphinx-build -b linkcheck . _build/lintcheck/
+# report will be generated in _build/lintcheck/output.txt
 
 Doxygen page Output
 ==



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] cbf778a - Remove also Block-ABI-Apple.txt from the Makefile

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-10-18T14:56:23+02:00
New Revision: cbf778a592fa5ee7c2bdd3d3ee7b468da2a7c9e6

URL: 
https://github.com/llvm/llvm-project/commit/cbf778a592fa5ee7c2bdd3d3ee7b468da2a7c9e6
DIFF: 
https://github.com/llvm/llvm-project/commit/cbf778a592fa5ee7c2bdd3d3ee7b468da2a7c9e6.diff

LOG: Remove also Block-ABI-Apple.txt from the Makefile

Added: 


Modified: 
clang/docs/Makefile.sphinx

Removed: 




diff  --git a/clang/docs/Makefile.sphinx b/clang/docs/Makefile.sphinx
index 7949e39c3687..06c3b86819d0 100644
--- a/clang/docs/Makefile.sphinx
+++ b/clang/docs/Makefile.sphinx
@@ -50,10 +50,6 @@ html:
@# Kind of a hack, but HTML-formatted docs are on the way out anyway.
@echo "Copying legacy HTML-formatted docs into $(BUILDDIR)/html"
@cp -a *.html $(BUILDDIR)/html
-   @# FIXME: What we really need is a way to specify redirects, so that
-   @# we can just redirect to a reST'ified version of this document.
-   @# PR14714 is tracking the issue of redirects.
-   @cp -a Block-ABI-Apple.txt $(BUILDDIR)/html
@echo "Build finished. The HTML pages are in $(BUILDDIR)/html."
 
 dirhtml:



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] b07ea8a - clang release notes: improve the wording

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-10-10T22:26:11+02:00
New Revision: b07ea8a967c5e77b42eeac9307fa006b7df87ca4

URL: 
https://github.com/llvm/llvm-project/commit/b07ea8a967c5e77b42eeac9307fa006b7df87ca4
DIFF: 
https://github.com/llvm/llvm-project/commit/b07ea8a967c5e77b42eeac9307fa006b7df87ca4.diff

LOG: clang release notes: improve the wording

Added: 


Modified: 
clang/docs/ReleaseNotes.rst

Removed: 




diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index e98a3fe00fa9..f32ea39a7718 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -51,7 +51,7 @@ Major New Features
 Improvements to Clang's diagnostics
 ^^^
 
-- -Wbitwise-instead-of-logical (part of -Wbool-operation) warns about use of 
bitwise with boolean operands which have side effects.
+- -Wbitwise-instead-of-logical (part of -Wbool-operation) warns about use of 
bitwise operators with boolean operands which have side effects.
 
 Non-comprehensive list of changes in this release
 -



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 9c8f950 - clang release notes: document the -Wbool-operation improvement

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-10-10T21:28:40+02:00
New Revision: 9c8f950a04004736bddb5093eda15f9a1c4f2eea

URL: 
https://github.com/llvm/llvm-project/commit/9c8f950a04004736bddb5093eda15f9a1c4f2eea
DIFF: 
https://github.com/llvm/llvm-project/commit/9c8f950a04004736bddb5093eda15f9a1c4f2eea.diff

LOG: clang release notes: document the -Wbool-operation improvement

Reviewed By: xbolva00

Differential Revision: https://reviews.llvm.org/D111215

Added: 


Modified: 
clang/docs/ReleaseNotes.rst

Removed: 




diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index dc8f8a3068245..e98a3fe00fa92 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -51,7 +51,7 @@ Major New Features
 Improvements to Clang's diagnostics
 ^^^
 
-- ...
+- -Wbitwise-instead-of-logical (part of -Wbool-operation) warns about use of 
bitwise with boolean operands which have side effects.
 
 Non-comprehensive list of changes in this release
 -



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] eccd477 - Add CMAKE_BUILD_TYPE to the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-09-21T10:44:08+02:00
New Revision: eccd477ce312c54e317876ca714c661325f3e318

URL: 
https://github.com/llvm/llvm-project/commit/eccd477ce312c54e317876ca714c661325f3e318
DIFF: 
https://github.com/llvm/llvm-project/commit/eccd477ce312c54e317876ca714c661325f3e318.diff

LOG: Add CMAKE_BUILD_TYPE to the list of BOOTSTRAP_DEFAULT_PASSTHROUGH variables

When building clang in stage2, when -DCMAKE_BUILD_TYPE=RelWithDebInfo is set,
the developer can expect that the stage2 clang is built using the same mode.
Especially as the performances are much worst in debug mode.
(Principle of least astonishment)

Differential Revision: https://reviews.llvm.org/D53014

Added: 


Modified: 
clang/CMakeLists.txt

Removed: 




diff  --git a/clang/CMakeLists.txt b/clang/CMakeLists.txt
index 29ceef8a4bd38..6e18e74c6294c 100644
--- a/clang/CMakeLists.txt
+++ b/clang/CMakeLists.txt
@@ -706,6 +706,7 @@ if (CLANG_ENABLE_BOOTSTRAP)
 CMAKE_CXX_COMPILER_LAUNCHER
 CMAKE_MAKE_PROGRAM
 CMAKE_OSX_ARCHITECTURES
+CMAKE_BUILD_TYPE
 LLVM_ENABLE_PROJECTS
 LLVM_ENABLE_RUNTIMES)
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fe611b1 - [clang] Move the soname declaration in a variable at the top of the file

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-08-27T09:07:12+02:00
New Revision: fe611b1da84b9442c093739394d336af9e99c1a1

URL: 
https://github.com/llvm/llvm-project/commit/fe611b1da84b9442c093739394d336af9e99c1a1
DIFF: 
https://github.com/llvm/llvm-project/commit/fe611b1da84b9442c093739394d336af9e99c1a1.diff

LOG: [clang] Move the soname declaration in a variable at the top of the file

Currently, it is a bit buried in the file even if this is
pretty important for distro.

Reviewed By: MaskRay

Differential Revision: https://reviews.llvm.org/D108533

Added: 


Modified: 
clang/tools/libclang/CMakeLists.txt

Removed: 




diff  --git a/clang/tools/libclang/CMakeLists.txt 
b/clang/tools/libclang/CMakeLists.txt
index 7148bdccfba48..bf88dca0a34b1 100644
--- a/clang/tools/libclang/CMakeLists.txt
+++ b/clang/tools/libclang/CMakeLists.txt
@@ -1,3 +1,9 @@
+# The SOVERSION should be updated only if a change is made to the libclang
+# ABI, and when it is updated, it should be updated to the current
+# LLVM_VERSION_MAJOR.
+# Please also see clang/tools/libclang/libclang.map
+set(CLANG_SONAME 13)
+
 set(SOURCES
   ARCMigrate.cpp
   BuildSystem.cpp
@@ -171,12 +177,9 @@ if(ENABLE_SHARED)
 set_property(SOURCE ARCMigrate.cpp APPEND PROPERTY
  OBJECT_DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/libclang.map)
 
-# The SOVERSION should be updated only if a change is made to the libclang
-# ABI, and when it is updated, it should be updated to the current
-# LLVM_VERSION_MAJOR.
 set_target_properties(libclang PROPERTIES
   VERSION 
${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH}${LLVM_VERSION_SUFFIX}
-  SOVERSION 13)
+  SOVERSION ${CLANG_SONAME})
   endif()
 endif()
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 70b06fe - scan-build-py: Force the opening in utf-8

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-08-17T19:19:39+02:00
New Revision: 70b06fe8a1862f5b9a0ef4e5e9098c1e457cf275

URL: 
https://github.com/llvm/llvm-project/commit/70b06fe8a1862f5b9a0ef4e5e9098c1e457cf275
DIFF: 
https://github.com/llvm/llvm-project/commit/70b06fe8a1862f5b9a0ef4e5e9098c1e457cf275.diff

LOG: scan-build-py: Force the opening in utf-8

It fails on ubuntu bionic otherwise with:
```
scan-build-py-14: Run 'scan-view 
/tmp/scan-build-2021-08-09-09-14-36-765350-nx9s888s' to examine bug reports.
scan-build-py-14: Internal error.
Traceback (most recent call last):
  File "/usr/lib/llvm-14/lib/libscanbuild/__init__.py", line 125, in wrapper
return function(*args, **kwargs)
  File "/usr/lib/llvm-14/lib/libscanbuild/analyze.py", line 72, in scan_build
number_of_bugs = document(args)
  File "/usr/lib/llvm-14/lib/libscanbuild/report.py", line 35, in document
for bug in read_bugs(args.output, html_reports_available):
  File "/usr/lib/llvm-14/lib/libscanbuild/report.py", line 282, in read_bugs
for bug in parser(bug_file):
  File "/usr/lib/llvm-14/lib/libscanbuild/report.py", line 421, in 
parse_bug_html
for line in handler.readlines():
  File "/usr/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 3360: 
ordinal not in range(128)
scan-build-py-14: Please run this command again and turn on verbose mode (add 
'-' as argument).
```

I guess it is caused by a problem in Python 3.6

Reviewed By: phosek, isthismyaccount

Differential Revision: https://reviews.llvm.org/D107887

Added: 


Modified: 
clang/tools/scan-build-py/lib/libscanbuild/report.py

Removed: 




diff  --git a/clang/tools/scan-build-py/lib/libscanbuild/report.py 
b/clang/tools/scan-build-py/lib/libscanbuild/report.py
index 729b25e6350f..0962b636a921 100644
--- a/clang/tools/scan-build-py/lib/libscanbuild/report.py
+++ b/clang/tools/scan-build-py/lib/libscanbuild/report.py
@@ -417,7 +417,7 @@ def parse_bug_html(filename):
 'bug_path_length': 1
 }
 
-with open(filename) as handler:
+with open(filename, encoding='utf-8') as handler:
 for line in handler.readlines():
 # do not read the file further
 if endsign.match(line):



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] b8d451d - Add support of the future Debian (Debian 12 - Bookworm)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-08-16T09:11:31+02:00
New Revision: b8d451da8610f0dd3ab55289606d7f2973e708d6

URL: 
https://github.com/llvm/llvm-project/commit/b8d451da8610f0dd3ab55289606d7f2973e708d6
DIFF: 
https://github.com/llvm/llvm-project/commit/b8d451da8610f0dd3ab55289606d7f2973e708d6.diff

LOG: Add support of the future Debian (Debian 12 - Bookworm)
https://wiki.debian.org/DebianBookworm

ETA: 2023

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 0d2a0939639ea..d9909bcf96968 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -37,6 +37,7 @@ class Distro {
 DebianStretch,
 DebianBuster,
 DebianBullseye,
+DebianBookworm,
 Exherbo,
 RHEL5,
 RHEL6,
@@ -119,7 +120,7 @@ class Distro {
   bool IsOpenSUSE() const { return DistroVal == OpenSUSE; }
 
   bool IsDebian() const {
-return DistroVal >= DebianLenny && DistroVal <= DebianBullseye;
+return DistroVal >= DebianLenny && DistroVal <= DebianBookworm;
   }
 
   bool IsUbuntu() const {

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index c4cf4e48b5b8d..cdb5a1725750f 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -150,6 +150,8 @@ static Distro::DistroType 
DetectDistro(llvm::vfs::FileSystem ) {
 return Distro::DebianBuster;
   case 11:
 return Distro::DebianBullseye;
+  case 12:
+return Distro::DebianBookworm;
   default:
 return Distro::UnknownDistro;
   }
@@ -161,6 +163,7 @@ static Distro::DistroType 
DetectDistro(llvm::vfs::FileSystem ) {
 .Case("stretch/sid", Distro::DebianStretch)
 .Case("buster/sid", Distro::DebianBuster)
 .Case("bullseye/sid", Distro::DebianBullseye)
+.Case("bookworm/sid", Distro::DebianBookworm)
 .Default(Distro::UnknownDistro);
   }
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] f6ba035 - [clang-tidy] ensure run-clang-tidy reports children killed by signals

[Sylvestre Ledru via cfe-commits]

Author: Ian Campbell
Date: 2021-07-19T14:18:26+02:00
New Revision: f6ba03584b3ce5dab45d8c1fe84e89d4d668574d

URL: 
https://github.com/llvm/llvm-project/commit/f6ba03584b3ce5dab45d8c1fe84e89d4d668574d
DIFF: 
https://github.com/llvm/llvm-project/commit/f6ba03584b3ce5dab45d8c1fe84e89d4d668574d.diff

LOG: [clang-tidy] ensure run-clang-tidy reports children killed by signals

If a clang-tidy child process exits with a signal then run-clang-tidy will exit
with an error but there is no hint why in the output, since the clang-tidy
doesn't log anything and may not even have had the opportunity to do so
depending on the signal used.

`subprocess.CompletedProcess.returncode` is the negative signal number in this
case.

I hit this in a CI system where the parallelism used exceeded the RAM assigned
to the container causing the OOM killer to SIGKILL clang-tidy processes.

Reviewed By: sylvestre.ledru

Differential Revision: https://reviews.llvm.org/D99081

Added: 


Modified: 
clang-tools-extra/clang-tidy/tool/run-clang-tidy.py

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py 
b/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
index de810230b2852..acd1ed6979c0d 100755
--- a/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
+++ b/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
@@ -173,6 +173,9 @@ def run_tidy(args, tmpdir, build_path, queue, lock, 
failed_files):
 proc = subprocess.Popen(invocation, stdout=subprocess.PIPE, 
stderr=subprocess.PIPE)
 output, err = proc.communicate()
 if proc.returncode != 0:
+  if proc.returncode < 0:
+msg = "%s: terminated by signal %d\n" % (name, -proc.returncode)
+err += msg.encode('utf-8')
   failed_files.append(name)
 with lock:
   sys.stdout.write(' '.join(invocation) + '\n' + output.decode('utf-8'))



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d71ee39 - Add support of the next Ubuntu (Ubuntu 21.10 - Impish Idri)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-04-22T20:38:28+02:00
New Revision: d71ee3993fe7decab00c3083438e13f81bd49f02

URL: 
https://github.com/llvm/llvm-project/commit/d71ee3993fe7decab00c3083438e13f81bd49f02
DIFF: 
https://github.com/llvm/llvm-project/commit/d71ee3993fe7decab00c3083438e13f81bd49f02.diff

LOG: Add support of the next Ubuntu (Ubuntu 21.10 - Impish Idri)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 0e98d00764c13..0d2a0939639ea 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -71,6 +71,7 @@ class Distro {
 UbuntuFocal,
 UbuntuGroovy,
 UbuntuHirsute,
+UbuntuImpish,
 UnknownDistro
   };
 
@@ -122,7 +123,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuHirsute;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuImpish;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 9fc790f17d6ce..c4cf4e48b5b8d 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -89,6 +89,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("focal", Distro::UbuntuFocal)
 .Case("groovy", Distro::UbuntuGroovy)
 .Case("hirsute", Distro::UbuntuHirsute)
+.Case("impish", Distro::UbuntuImpish)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 56cb214 - add test case for ignoring -flto=auto and -flto=jobserver

[Sylvestre Ledru via cfe-commits]

Author: Matthias Klose
Date: 2021-04-15T12:19:14+02:00
New Revision: 56cb214b38a70c0fc824ab096e500d78c72b44ef

URL: 
https://github.com/llvm/llvm-project/commit/56cb214b38a70c0fc824ab096e500d78c72b44ef
DIFF: 
https://github.com/llvm/llvm-project/commit/56cb214b38a70c0fc824ab096e500d78c72b44ef.diff

LOG: add test case for ignoring -flto=auto and -flto=jobserver

as requested in https://reviews.llvm.org/D99501, test that the two new options 
are ignored.

Reviewed By: tejohnson, fhahn

Differential Revision: https://reviews.llvm.org/D100484

Added: 


Modified: 
clang/test/Driver/clang_f_opts.c

Removed: 




diff  --git a/clang/test/Driver/clang_f_opts.c 
b/clang/test/Driver/clang_f_opts.c
index 3ea9410adbead..9da9f5eac6324 100644
--- a/clang/test/Driver/clang_f_opts.c
+++ b/clang/test/Driver/clang_f_opts.c
@@ -279,6 +279,7 @@
 // RUN: -fno-delete-null-pointer-checks -fdelete-null-pointer-checks  \
 // RUN: -fno-inline-small-functions -finline-small-functions  \
 // RUN: -fno-fat-lto-objects -ffat-lto-objects\
+// RUN: -flto=auto -flto=jobserver\
 // RUN: -fno-merge-constants -fmerge-constants\
 // RUN: -fno-caller-saves -fcaller-saves  \
 // RUN: -fno-reorder-blocks -freorder-blocks  \



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 1628486 - ignore -flto= options recognized by GCC

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2021-04-05T11:54:17+02:00
New Revision: 1628486548420f85b3467026d54663d1516404f5

URL: 
https://github.com/llvm/llvm-project/commit/1628486548420f85b3467026d54663d1516404f5
DIFF: 
https://github.com/llvm/llvm-project/commit/1628486548420f85b3467026d54663d1516404f5.diff

LOG: ignore -flto= options recognized by GCC

as requested in https://bugs.llvm.org/show_bug.cgi?id=49553, submitting the 
proposed changes to just ignore the -flto= options which are recognized by GCC 
("auto" and "jobserver").

GCC supports -flto= to select the parallelity for LTO 
builds. LLVM also has -flto-jobs=, which only seems to have a meaning when 
used with -flto=thin?

The attached patch just ignores the values "auto" and "jobserver". that doesn't 
change anything in functionality.  Another option would be to map these values 
to either "thin" or "full", maybe in presence of the -ffat-lto-objects option?

-flto= could also be translated to -flto-jobs=.

Reviewed By: tejohnson

Differential Revision: https://reviews.llvm.org/D99501

Added: 


Modified: 
clang/include/clang/Driver/Options.td

Removed: 




diff  --git a/clang/include/clang/Driver/Options.td 
b/clang/include/clang/Driver/Options.td
index a6583acd9ecb..64d658b2bfd9 100644
--- a/clang/include/clang/Driver/Options.td
+++ b/clang/include/clang/Driver/Options.td
@@ -4155,6 +4155,8 @@ defm reorder_blocks : BooleanFFlag<"reorder-blocks">, 
Group, 
Group;
 defm default_inline : BooleanFFlag<"default-inline">, 
Group;
 defm fat_lto_objects : BooleanFFlag<"fat-lto-objects">, 
Group;
+def : Flag<["-"], "flto=auto">, Group;
+def : Flag<["-"], "flto=jobserver">, 
Group;
 defm float_store : BooleanFFlag<"float-store">, 
Group;
 defm friend_injection : BooleanFFlag<"friend-injection">, 
Group;
 defm function_attribute_list : BooleanFFlag<"function-attribute-list">, 
Group;



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 4d59c8f - -fstack-clash-protection: Return an actual error when used on unsupported OS

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-12-22T12:06:08+01:00
New Revision: 4d59c8fdb955ea0d668b854f467e12bce05a8857

URL: 
https://github.com/llvm/llvm-project/commit/4d59c8fdb955ea0d668b854f467e12bce05a8857
DIFF: 
https://github.com/llvm/llvm-project/commit/4d59c8fdb955ea0d668b854f467e12bce05a8857.diff

LOG: -fstack-clash-protection: Return an actual error when used on unsupported 
OS

$ clang-12: error: -fstack-clash-protection is not supported on Windows or Mac 
OS X

Differential Revision: https://reviews.llvm.org/D92245

Added: 


Modified: 
clang/include/clang/Basic/DiagnosticDriverKinds.td
clang/lib/Driver/ToolChains/Clang.cpp
llvm/test/CodeGen/X86/stack-clash-large.ll

Removed: 




diff  --git a/clang/include/clang/Basic/DiagnosticDriverKinds.td 
b/clang/include/clang/Basic/DiagnosticDriverKinds.td
index e92a4bf1dac5..736950b0abb1 100644
--- a/clang/include/clang/Basic/DiagnosticDriverKinds.td
+++ b/clang/include/clang/Basic/DiagnosticDriverKinds.td
@@ -273,6 +273,8 @@ def err_drv_unsupported_embed_bitcode
 : Error<"%0 is not supported with -fembed-bitcode">;
 def err_drv_bitcode_unsupported_on_toolchain : Error<
   "-fembed-bitcode is not supported on versions of iOS prior to 6.0">;
+def err_drv_stack_clash_protection_unsupported_on_toolchain : Error<
+  "-fstack-clash-protection is not supported on %0">;
 
 def err_drv_invalid_malign_branch_EQ : Error<
   "invalid argument '%0' to -malign-branch=; each element must be one of: %1">;

diff  --git a/clang/lib/Driver/ToolChains/Clang.cpp 
b/clang/lib/Driver/ToolChains/Clang.cpp
index 300ab6e815e2..c04b350dae7d 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -3067,12 +3067,15 @@ static void RenderSSPOptions(const Driver , const 
ToolChain ,
   }
 }
 
-static void RenderSCPOptions(const ToolChain , const ArgList ,
- ArgStringList ) {
+static void RenderSCPOptions(const Driver , const ToolChain ,
+ const ArgList , ArgStringList ) {
   const llvm::Triple  = TC.getEffectiveTriple();
 
-  if (!EffectiveTriple.isOSLinux())
+  if (EffectiveTriple.isOSWindows()) {
+D.Diag(diag::err_drv_stack_clash_protection_unsupported_on_toolchain)
+<< EffectiveTriple.getOSName();
 return;
+  }
 
   if (!EffectiveTriple.isX86() && !EffectiveTriple.isSystemZ() &&
   !EffectiveTriple.isPPC64())
@@ -5550,7 +5553,7 @@ void Clang::ConstructJob(Compilation , const JobAction 
,
 CmdArgs.push_back(Args.MakeArgString("-mspeculative-load-hardening"));
 
   RenderSSPOptions(D, TC, Args, CmdArgs, KernelOrKext);
-  RenderSCPOptions(TC, Args, CmdArgs);
+  RenderSCPOptions(D, TC, Args, CmdArgs);
   RenderTrivialAutoVarInitOptions(D, TC, Args, CmdArgs);
 
   // Translate -mstackrealign

diff  --git a/llvm/test/CodeGen/X86/stack-clash-large.ll 
b/llvm/test/CodeGen/X86/stack-clash-large.ll
index dd53cd8f6964..7deae310f617 100644
--- a/llvm/test/CodeGen/X86/stack-clash-large.ll
+++ b/llvm/test/CodeGen/X86/stack-clash-large.ll
@@ -1,5 +1,7 @@
 ; RUN: llc -mtriple=x86_64-linux-android < %s | FileCheck 
-check-prefix=CHECK-X86-64 %s 
 ; RUN: llc -mtriple=i686-linux-android < %s | FileCheck 
-check-prefix=CHECK-X86-32 %s 
+; RUN: llc -mtriple=x86_64-unknown-freebsd < %s | FileCheck 
-check-prefix=CHECK-X86-64 %s
+; RUN: llc -mtriple=x86_64-pc-linux-gnu < %s | FileCheck 
-check-prefix=CHECK-X86-64 %s
 
 define i32 @foo() local_unnamed_addr #0 {
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 43e451f - Fix gendered documentation

[Sylvestre Ledru via cfe-commits]

Author: Pedro Gonnet
Date: 2020-10-31T12:17:19+01:00
New Revision: 43e451f9f316fd224d88899d45313a73ff11291e

URL: 
https://github.com/llvm/llvm-project/commit/43e451f9f316fd224d88899d45313a73ff11291e
DIFF: 
https://github.com/llvm/llvm-project/commit/43e451f9f316fd224d88899d45313a73ff11291e.diff

LOG: Fix gendered documentation

Changed two references to developers as "he" or "him" to the more neutral 
"they".

Reviewed By: JDevlieghere, sylvestre.ledru

Differential Revision: https://reviews.llvm.org/D78807

Added: 


Modified: 

clang-tools-extra/docs/clang-tidy/checks/google-objc-global-variable-declaration.rst
lldb/docs/use/variable.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/google-objc-global-variable-declaration.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/google-objc-global-variable-declaration.rst
index 15b59996e3d3..84c97767f18f 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/google-objc-global-variable-declaration.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/google-objc-global-variable-declaration.rst
@@ -44,4 +44,4 @@ However for code that prefixed with non-alphabetical 
characters like:
   static NSString* __anotherString = @"world";
 
 The check will give a warning message but will not be able to suggest a fix. 
The
-user need to fix it on his own.
+user needs to fix it on their own.

diff  --git a/lldb/docs/use/variable.rst b/lldb/docs/use/variable.rst
index 5b1fef7b06c3..ae6dbf2f4e5e 100644
--- a/lldb/docs/use/variable.rst
+++ b/lldb/docs/use/variable.rst
@@ -993,7 +993,7 @@ have many member variables but not all of these are 
actually necessary for the
 user to see.
 
 A filter will solve this issue by only letting the user see those member
-variables he cares about. Of course, the equivalent of a filter can be
+variables they care about. Of course, the equivalent of a filter can be
 implemented easily using synthetic children, but a filter lets you get the job
 done without having to write Python code.
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 5d1eae7 - Add support of the next Ubuntu (Ubuntu 21.04 - Hirsute Hippo)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-10-29T19:11:32+01:00
New Revision: 5d1eae7d23e3477d6044cf9d60a90ce9719f17e6

URL: 
https://github.com/llvm/llvm-project/commit/5d1eae7d23e3477d6044cf9d60a90ce9719f17e6
DIFF: 
https://github.com/llvm/llvm-project/commit/5d1eae7d23e3477d6044cf9d60a90ce9719f17e6.diff

LOG: Add support of the next Ubuntu (Ubuntu 21.04 - Hirsute Hippo)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 2b5ec9ab8eea..0e98d00764c1 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -70,6 +70,7 @@ class Distro {
 UbuntuEoan,
 UbuntuFocal,
 UbuntuGroovy,
+UbuntuHirsute,
 UnknownDistro
   };
 
@@ -121,7 +122,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuGroovy;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuHirsute;
   }
 
   bool IsAlpineLinux() const { return DistroVal == AlpineLinux; }

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index 02a71cbd5a9c..ee4fe841e7ee 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -87,6 +87,7 @@ static Distro::DistroType 
DetectLsbRelease(llvm::vfs::FileSystem ) {
 .Case("eoan", Distro::UbuntuEoan)
 .Case("focal", Distro::UbuntuFocal)
 .Case("groovy", Distro::UbuntuGroovy)
+.Case("hirsute", Distro::UbuntuHirsute)
 .Default(Distro::UnknownDistro);
   return Version;
 }



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] b9e7894 - Revert "[clang-format] Fix AlignConsecutive on PP blocks"

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-10-17T19:52:51+02:00
New Revision: b9e789447f14c0330edd22c82746af29e7c3b259

URL: 
https://github.com/llvm/llvm-project/commit/b9e789447f14c0330edd22c82746af29e7c3b259
DIFF: 
https://github.com/llvm/llvm-project/commit/b9e789447f14c0330edd22c82746af29e7c3b259.diff

LOG: Revert "[clang-format] Fix AlignConsecutive on PP blocks"

This reverts commit b2eb439317576ce718193763c12bff9fccdfc166.

Caused the regression:
https://bugs.llvm.org/show_bug.cgi?id=47589

Reviewed By: MyDeveloperDay

Differential Revision: https://reviews.llvm.org/D89464

Added: 


Modified: 
clang/lib/Format/FormatToken.h
clang/lib/Format/UnwrappedLineParser.cpp
clang/lib/Format/WhitespaceManager.cpp
clang/unittests/Format/FormatTest.cpp
clang/unittests/Format/FormatTestComments.cpp

Removed: 




diff  --git a/clang/lib/Format/FormatToken.h b/clang/lib/Format/FormatToken.h
index 9cc65bb11b54..a3cb81f1b1ef 100644
--- a/clang/lib/Format/FormatToken.h
+++ b/clang/lib/Format/FormatToken.h
@@ -206,12 +206,11 @@ class AnnotatedLine;
 struct FormatToken {
   FormatToken()
   : HasUnescapedNewline(false), IsMultiline(false), IsFirst(false),
-MustBreakBefore(false), MustBreakAlignBefore(false),
-IsUnterminatedLiteral(false), CanBreakBefore(false),
-ClosesTemplateDeclaration(false), StartsBinaryExpression(false),
-EndsBinaryExpression(false), PartOfMultiVariableDeclStmt(false),
-ContinuesLineCommentSection(false), Finalized(false),
-BlockKind(BK_Unknown), Decision(FD_Unformatted),
+MustBreakBefore(false), IsUnterminatedLiteral(false),
+CanBreakBefore(false), ClosesTemplateDeclaration(false),
+StartsBinaryExpression(false), EndsBinaryExpression(false),
+PartOfMultiVariableDeclStmt(false), ContinuesLineCommentSection(false),
+Finalized(false), BlockKind(BK_Unknown), Decision(FD_Unformatted),
 PackingKind(PPK_Inconclusive), Type(TT_Unknown) {}
 
   /// The \c Token.
@@ -248,12 +247,6 @@ struct FormatToken {
   /// before the token.
   unsigned MustBreakBefore : 1;
 
-  /// Whether to not align across this token
-  ///
-  /// This happens for example when a preprocessor directive ended directly
-  /// before the token, but very rarely otherwise.
-  unsigned MustBreakAlignBefore : 1;
-
   /// Set to \c true if this token is an unterminated literal.
   unsigned IsUnterminatedLiteral : 1;
 

diff  --git a/clang/lib/Format/UnwrappedLineParser.cpp 
b/clang/lib/Format/UnwrappedLineParser.cpp
index ab1f647481d0..044f034013dc 100644
--- a/clang/lib/Format/UnwrappedLineParser.cpp
+++ b/clang/lib/Format/UnwrappedLineParser.cpp
@@ -3046,7 +3046,6 @@ void UnwrappedLineParser::readToken(int LevelDifference) {
   }
   FormatTok = Tokens->getNextToken();
   FormatTok->MustBreakBefore = true;
-  FormatTok->MustBreakAlignBefore = true;
 }
 
 if (!PPStack.empty() && (PPStack.back().Kind == PP_Unreachable) &&
@@ -3071,7 +3070,6 @@ void UnwrappedLineParser::pushToken(FormatToken *Tok) {
   Line->Tokens.push_back(UnwrappedLineNode(Tok));
   if (MustBreakBeforeNextToken) {
 Line->Tokens.back().Tok->MustBreakBefore = true;
-Line->Tokens.back().Tok->MustBreakAlignBefore = true;
 MustBreakBeforeNextToken = false;
   }
 }

diff  --git a/clang/lib/Format/WhitespaceManager.cpp 
b/clang/lib/Format/WhitespaceManager.cpp
index 2d479817118d..9e47b49da3d0 100644
--- a/clang/lib/Format/WhitespaceManager.cpp
+++ b/clang/lib/Format/WhitespaceManager.cpp
@@ -411,11 +411,9 @@ static unsigned AlignTokens(const FormatStyle , F 
&,
 if (Changes[i].NewlinesBefore != 0) {
   CommasBeforeMatch = 0;
   EndOfSequence = i;
-  // If there is a blank line, there is a forced-align-break (eg,
-  // preprocessor), or if the last line didn't contain any matching token,
-  // the sequence ends here.
-  if (Changes[i].NewlinesBefore > 1 ||
-  Changes[i].Tok->MustBreakAlignBefore || !FoundMatchOnLine)
+  // If there is a blank line, or if the last line didn't contain any
+  // matching token, the sequence ends here.
+  if (Changes[i].NewlinesBefore > 1 || !FoundMatchOnLine)
 AlignCurrentSequence();
 
   FoundMatchOnLine = false;
@@ -726,8 +724,6 @@ void WhitespaceManager::alignTrailingComments() {
 if (Changes[i].StartOfBlockComment)
   continue;
 Newlines += Changes[i].NewlinesBefore;
-if (Changes[i].Tok->MustBreakAlignBefore)
-  BreakBeforeNext = true;
 if (!Changes[i].IsTrailingComment)
   continue;
 

diff  --git a/clang/unittests/Format/FormatTest.cpp 
b/clang/unittests/Format/FormatTest.cpp
index 27e76d40d125..7686e252f7c2 100644
--- a/clang/unittests/Format/FormatTest.cpp
+++ b/clang/unittests/Format/FormatTest.cpp
@@ -12254,26 +12254,28 @@ TEST_F(FormatTest, AlignConsecutiveAssignments) {
Alignment);
 
   // Bug 25167
- 

[clang-tools-extra] 002968a - [clang-tidy] Add an example for misc-unused-alias-decls

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-10-13T13:56:04+02:00
New Revision: 002968a320463237f0ccea754a7482b229e34cf1

URL: 
https://github.com/llvm/llvm-project/commit/002968a320463237f0ccea754a7482b229e34cf1
DIFF: 
https://github.com/llvm/llvm-project/commit/002968a320463237f0ccea754a7482b229e34cf1.diff

LOG: [clang-tidy] Add an example for misc-unused-alias-decls

Differential Revision: https://reviews.llvm.org/D89270

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/misc-unused-alias-decls.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-alias-decls.rst 
b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-alias-decls.rst
index d0e8c7188a3f..1b0a0cde059b 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-alias-decls.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-alias-decls.rst
@@ -5,3 +5,10 @@ misc-unused-alias-decls
 
 
 Finds unused namespace alias declarations.
+
+.. code-block:: c++
+
+  namespace my_namespace {
+  class C {};
+  }
+  namespace unused_alias = ::my_namespace;



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] bbea4d5 - clang: Don't show a trailing space with --version when not built from the repo

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-07-08T14:02:02+02:00
New Revision: bbea4d5e6b82a683dccaa8f4916e2a44f5dd3490

URL: 
https://github.com/llvm/llvm-project/commit/bbea4d5e6b82a683dccaa8f4916e2a44f5dd3490
DIFF: 
https://github.com/llvm/llvm-project/commit/bbea4d5e6b82a683dccaa8f4916e2a44f5dd3490.diff

LOG: clang: Don't show a trailing space with --version when not built from the 
repo

Reported here:
https://bugs.llvm.org/show_bug.cgi?id=38998#c15

Reviewers: hans

Differential Revision: https://reviews.llvm.org/D83386

Added: 


Modified: 
clang/lib/Basic/Version.cpp

Removed: 




diff  --git a/clang/lib/Basic/Version.cpp b/clang/lib/Basic/Version.cpp
index c4b7d34ed168..286107cab9d7 100644
--- a/clang/lib/Basic/Version.cpp
+++ b/clang/lib/Basic/Version.cpp
@@ -97,8 +97,12 @@ std::string getClangToolFullVersion(StringRef ToolName) {
 #ifdef CLANG_VENDOR
   OS << CLANG_VENDOR;
 #endif
-  OS << ToolName << " version " CLANG_VERSION_STRING " "
- << getClangFullRepositoryVersion();
+  OS << ToolName << " version " CLANG_VERSION_STRING;
+
+  std::string repo = getClangFullRepositoryVersion();
+  if (!repo.empty()) {
+OS << " " << repo;
+  }
 
   return OS.str();
 }
@@ -111,7 +115,13 @@ std::string getClangFullCPPVersion() {
 #ifdef CLANG_VENDOR
   OS << CLANG_VENDOR;
 #endif
-  OS << "Clang " CLANG_VERSION_STRING " " << getClangFullRepositoryVersion();
+  OS << "Clang " CLANG_VERSION_STRING;
+
+  std::string repo = getClangFullRepositoryVersion();
+  if (!repo.empty()) {
+OS << " " << repo;
+  }
+
   return OS.str();
 }
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] a341863 - libclc: update website url

[Sylvestre Ledru via cfe-commits]

Author: davidak
Date: 2020-05-29T09:18:37+02:00
New Revision: a3418631e8aa0941b8b57ec2fc3b8d0c7db493be

URL: 
https://github.com/llvm/llvm-project/commit/a3418631e8aa0941b8b57ec2fc3b8d0c7db493be
DIFF: 
https://github.com/llvm/llvm-project/commit/a3418631e8aa0941b8b57ec2fc3b8d0c7db493be.diff

LOG: libclc: update website url

old link is dead

Added: 


Modified: 
libclc/README.TXT

Removed: 




diff  --git a/libclc/README.TXT b/libclc/README.TXT
index 00ae6bfa40a1..57b5242b9bbe 100644
--- a/libclc/README.TXT
+++ b/libclc/README.TXT
@@ -49,4 +49,4 @@ $ DESTDIR=/path/for/staged/install ninja install
 Website
 ---
 
-http://www.pcc.me.uk/~peter/libclc/
+https://libclc.llvm.org/



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] ca46754 - [CMake] Pass CLANG_VENDOR variables into later stages

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-29T09:13:08+02:00
New Revision: ca467542eecfc621eea7fefb3c7e3849c6b43ac7

URL: 
https://github.com/llvm/llvm-project/commit/ca467542eecfc621eea7fefb3c7e3849c6b43ac7
DIFF: 
https://github.com/llvm/llvm-project/commit/ca467542eecfc621eea7fefb3c7e3849c6b43ac7.diff

LOG: [CMake] Pass CLANG_VENDOR variables into later stages

We are already passing CLANG_VERSION_* & PACKAGE_VENDOR

Added: 


Modified: 
clang/CMakeLists.txt

Removed: 




diff  --git a/clang/CMakeLists.txt b/clang/CMakeLists.txt
index 7dadc5f6e917..5a5e34aacbeb 100644
--- a/clang/CMakeLists.txt
+++ b/clang/CMakeLists.txt
@@ -711,6 +711,7 @@ if (CLANG_ENABLE_BOOTSTRAP)
 CLANG_VERSION_MAJOR
 CLANG_VERSION_MINOR
 CLANG_VERSION_PATCHLEVEL
+CLANG_VENDOR
 LLVM_VERSION_SUFFIX
 LLVM_BINUTILS_INCDIR
 CLANG_REPOSITORY_STRING



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] dbb0349 - Add support of the next Ubuntu (Ubuntu 20.10 - Groovy Gorilla)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-19T08:30:25+02:00
New Revision: dbb034947cc16d26164be1e78d95249d60738dc6

URL: 
https://github.com/llvm/llvm-project/commit/dbb034947cc16d26164be1e78d95249d60738dc6
DIFF: 
https://github.com/llvm/llvm-project/commit/dbb034947cc16d26164be1e78d95249d60738dc6.diff

LOG: Add support of the next Ubuntu (Ubuntu 20.10 - Groovy Gorilla)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index d382cf77a8b2..038d4ce75d80 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -67,6 +67,7 @@ class Distro {
 UbuntuDisco,
 UbuntuEoan,
 UbuntuFocal,
+UbuntuGroovy,
 UnknownDistro
   };
 
@@ -120,7 +121,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuFocal;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuGroovy;
   }
 
   bool IsAlpineLinux() const {

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index acfde7d8bd0b..4d58ad1ae78c 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -71,6 +71,7 @@ static Distro::DistroType DetectDistro(llvm::vfs::FileSystem 
,
   .Case("disco", Distro::UbuntuDisco)
   .Case("eoan", Distro::UbuntuEoan)
   .Case("focal", Distro::UbuntuFocal)
+  .Case("groovy", Distro::UbuntuGroovy)
   .Default(Distro::UnknownDistro);
 if (Version != Distro::UnknownDistro)
   return Version;



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] a832fc4 - Revert two patches, not ready to be shared

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-10T11:27:13+02:00
New Revision: a832fc4b0c1e056edbb458973fac1f0c6db7a9e5

URL: 
https://github.com/llvm/llvm-project/commit/a832fc4b0c1e056edbb458973fac1f0c6db7a9e5
DIFF: 
https://github.com/llvm/llvm-project/commit/a832fc4b0c1e056edbb458973fac1f0c6db7a9e5.diff

LOG: Revert two patches, not ready to be shared

Revert "clang-tidy doc: add a note for checkers with an autofix"

This reverts commit dc0f79ea5b99b9609a21c29089c294ab82d88094.

Revert "add_new_check.py: Update of the template to add an autofix section"

This reverts commit f97f92e5b0fd14240c2ec27c44e1b3cfddfdf668.

Added: 


Modified: 
clang-tools-extra/clang-tidy/add_new_check.py
clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst

clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/add_new_check.py 
b/clang-tools-extra/clang-tidy/add_new_check.py
index fa8b4da4d624..4477444cc127 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -391,30 +391,6 @@ def write_docs(module_path, module, check_name):
 %(underline)s
 
 FIXME: Describe what patterns does the check detect and why. Give examples.
-
-FIXME: Remove of the following section:
-
-Automatic Fixes
-^^^
-
-The check doesn't provide an automatic fix.
-
-or
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes. See the documentation above for the 
details.
-
-or
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes in the following cases:
-1.
-2.
-3.
 """ % {'check_name_dashes': check_name_dashes,
'underline': '=' * len(check_name_dashes)})
 

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst 
b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
index de520e9511c7..3dfeb299de06 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
@@ -40,8 +40,3 @@ Options
constructors - no constructor initializers). When the function body is 
empty,
an unused parameter is unlikely to be unnoticed by a human reader, and
there's basically no place for a bug to hide.
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes. See the documentation above for the 
details.

diff  --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
index 35be77d11c08..82c290e4a21b 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
@@ -86,8 +86,3 @@ that accepts 2 arguments. With permissive parameter list, it 
instead generates
 which is correct.
   
 This check requires using C++14 or higher to run.
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
index a36d58e4e31d..f217ff090445 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
@@ -15,8 +15,3 @@ Deprecated member type   Replacement
 ``std::ios_base::streamoff``  
 ``std::ios_base::streampos`` 
 ===  ===
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
index 89d4a0c95a01..82b27bbe9020 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
@@ -265,8 +265,3 @@ exclude incorrect diagnostics on OpenMP for loops prior to 
OpenMP 5.
 To prevent this check to be applied (and to break) OpenMP for loops but still 
be
 applied to non-OpenMP for loops the usage of ``NOLINT`` (see
 :ref:`clang-tidy-nolint`) on the specific for loops is recommended.
-
-Automatic Fixes
-^^^
-
-The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 

[clang-tools-extra] f97f92e - add_new_check.py: Update of the template to add an autofix section

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-10T11:25:19+02:00
New Revision: f97f92e5b0fd14240c2ec27c44e1b3cfddfdf668

URL: 
https://github.com/llvm/llvm-project/commit/f97f92e5b0fd14240c2ec27c44e1b3cfddfdf668
DIFF: 
https://github.com/llvm/llvm-project/commit/f97f92e5b0fd14240c2ec27c44e1b3cfddfdf668.diff

LOG: add_new_check.py: Update of the template to add an autofix section

Summary: As discussed here: https://reviews.llvm.org/D77983

Reviewers: alexfh

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D78052

Added: 


Modified: 
clang-tools-extra/clang-tidy/add_new_check.py

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/add_new_check.py 
b/clang-tools-extra/clang-tidy/add_new_check.py
index 4477444cc127..fa8b4da4d624 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -391,6 +391,30 @@ def write_docs(module_path, module, check_name):
 %(underline)s
 
 FIXME: Describe what patterns does the check detect and why. Give examples.
+
+FIXME: Remove of the following section:
+
+Automatic Fixes
+^^^
+
+The check doesn't provide an automatic fix.
+
+or
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.
+
+or
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes in the following cases:
+1.
+2.
+3.
 """ % {'check_name_dashes': check_name_dashes,
'underline': '=' * len(check_name_dashes)})
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] b082813 - clang-tidy doc: Fix the syntax to use rst list type

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-10T11:25:20+02:00
New Revision: b0828135dbe3ee52a15163856008bcf05e81ccf8

URL: 
https://github.com/llvm/llvm-project/commit/b0828135dbe3ee52a15163856008bcf05e81ccf8
DIFF: 
https://github.com/llvm/llvm-project/commit/b0828135dbe3ee52a15163856008bcf05e81ccf8.diff

LOG: clang-tidy doc: Fix the syntax to use rst list type

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/bugprone-reserved-identifier.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/bugprone-reserved-identifier.rst 
b/clang-tools-extra/docs/clang-tidy/checks/bugprone-reserved-identifier.rst
index 203b1f4df9c1..388acf87efc7 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/bugprone-reserved-identifier.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/bugprone-reserved-identifier.rst
@@ -8,8 +8,9 @@ bugprone-reserved-identifier
 Checks for usages of identifiers reserved for use by the implementation. 
 
 The C and C++ standards both reserve the following names for such use:
-* identifiers that begin with an underscore followed by an uppercase letter;
-* identifiers in the global namespace that begin with an underscore.
+
+- identifiers that begin with an underscore followed by an uppercase letter;
+- identifiers in the global namespace that begin with an underscore.
 
 The C standard additionally reserves names beginning with a double underscore,
 while the C++ standard strengthens this to reserve names with a double 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] dc0f79e - clang-tidy doc: add a note for checkers with an autofix

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-05-10T11:25:19+02:00
New Revision: dc0f79ea5b99b9609a21c29089c294ab82d88094

URL: 
https://github.com/llvm/llvm-project/commit/dc0f79ea5b99b9609a21c29089c294ab82d88094
DIFF: 
https://github.com/llvm/llvm-project/commit/dc0f79ea5b99b9609a21c29089c294ab82d88094.diff

LOG: clang-tidy doc: add a note for checkers with an autofix

Summary:
Currently, when looking at a checker documentation, we have to go back
to the whole list or look at the sources to figure out if an autofix
is available or not.

Reviewers: alexfh, aaron.ballman

Subscribers: wuzish, Eugene.Zelenko, nemanjai, kbarton, arphaman, Charusso, 
cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D77983

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst

clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst 
b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
index 3dfeb299de06..de520e9511c7 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
@@ -40,3 +40,8 @@ Options
constructors - no constructor initializers). When the function body is 
empty,
an unused parameter is unlikely to be unnoticed by a human reader, and
there's basically no place for a bug to hide.
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.

diff  --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
index 82c290e4a21b..35be77d11c08 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
@@ -86,3 +86,8 @@ that accepts 2 arguments. With permissive parameter list, it 
instead generates
 which is correct.
   
 This check requires using C++14 or higher to run.
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
index f217ff090445..a36d58e4e31d 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
@@ -15,3 +15,8 @@ Deprecated member type   Replacement
 ``std::ios_base::streamoff``  
 ``std::ios_base::streampos`` 
 ===  ===
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
index 82b27bbe9020..89d4a0c95a01 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
@@ -265,3 +265,8 @@ exclude incorrect diagnostics on OpenMP for loops prior to 
OpenMP 5.
 To prevent this check to be applied (and to break) OpenMP for loops but still 
be
 applied to non-OpenMP for loops the usage of ``NOLINT`` (see
 :ref:`clang-tidy-nolint`) on the specific for loops is recommended.
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
index e73b32034f3d..f81ac2c9cbb9 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
@@ -48,3 +48,8 @@ Options
 
If set to non-zero, the check will not give warnings inside macros. Default
is `1`.
+
+Automatic Fixes
+^^^
+
+The check provides suggested fixes. See the documentation above for the 
details.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst 
b/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst
index 792d3a4c3dc4..09556800ed86 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst

[clang] 68e89c5 - [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu

[Sylvestre Ledru via cfe-commits]

Author: Samuel Thibault
Date: 2020-05-01T12:37:12+02:00
New Revision: 68e89c5b96038e00aeeffa642c415cfc0bcaa3dc

URL: 
https://github.com/llvm/llvm-project/commit/68e89c5b96038e00aeeffa642c415cfc0bcaa3dc
DIFF: 
https://github.com/llvm/llvm-project/commit/68e89c5b96038e00aeeffa642c415cfc0bcaa3dc.diff

LOG: [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu
Add missing files

Summary: The current code for GNU/Linux is actually completely generic, and can 
be moved to Gnu, so it can benefit GNU/Hurd and GNU/kFreeBSD

Reviewers: kristina, sammccall, lebedev.ri, MaskRay, arsenm, phosek

Reviewed By: MaskRay, phosek

Subscribers: wdng, ormris, emaste, arichardson, krytarowski, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D73845

Added: 
clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/as
clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/ld
clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/lib/.keep

clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o

clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o

clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginS.o

clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginT.o

Modified: 


Removed: 




diff  --git 
a/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/as 
b/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/as
new file mode 100755
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/ld 
b/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/bin/ld
new file mode 100755
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/lib/.keep 
b/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/i386-gnu/lib/.keep
new file mode 100644
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o
 
b/clang/test/Driver/Inputs/basic_cross_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o
new file mode 100644
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o
 
b/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbegin.o
new file mode 100644
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginS.o
 
b/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginS.o
new file mode 100644
index ..e69de29bb2d1

diff  --git 
a/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginT.o
 
b/clang/test/Driver/Inputs/basic_hurd_tree/usr/lib/gcc/i386-gnu/4.6.0/crtbeginT.o
new file mode 100644
index ..e69de29bb2d1



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] c298e5a - [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu

[Sylvestre Ledru via cfe-commits]

Author: Samuel Thibault
Date: 2020-05-01T12:12:27+02:00
New Revision: c298e5a022928b59941d74680a41fc9c76038b2a

URL: 
https://github.com/llvm/llvm-project/commit/c298e5a022928b59941d74680a41fc9c76038b2a
DIFF: 
https://github.com/llvm/llvm-project/commit/c298e5a022928b59941d74680a41fc9c76038b2a.diff

LOG: [Gnu toolchain] Move GCC multilib/multiarch paths support from Linux to Gnu

Summary: The current code for GNU/Linux is actually completely generic, and can 
be moved to Gnu, so it can benefit GNU/Hurd and GNU/kFreeBSD

Reviewers: kristina, sammccall, lebedev.ri, MaskRay, arsenm, phosek

Reviewed By: MaskRay, phosek

Subscribers: wdng, ormris, emaste, arichardson, krytarowski, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D73845

Added: 


Modified: 
clang/lib/Driver/ToolChains/Hurd.cpp
clang/test/Driver/hurd.c

Removed: 




diff  --git a/clang/lib/Driver/ToolChains/Hurd.cpp 
b/clang/lib/Driver/ToolChains/Hurd.cpp
index ce1806c4043b..3448e4f4a294 100644
--- a/clang/lib/Driver/ToolChains/Hurd.cpp
+++ b/clang/lib/Driver/ToolChains/Hurd.cpp
@@ -63,7 +63,19 @@ static StringRef getOSLibDir(const llvm::Triple , 
const ArgList ) {
 
 Hurd::Hurd(const Driver , const llvm::Triple , const ArgList )
 : Generic_ELF(D, Triple, Args) {
+  GCCInstallation.init(Triple, Args);
+  Multilibs = GCCInstallation.getMultilibs();
+  SelectedMultilib = GCCInstallation.getMultilib();
   std::string SysRoot = computeSysRoot();
+  ToolChain::path_list  = getProgramPaths();
+
+  Generic_GCC::PushPPaths(PPaths);
+
+  // The selection of paths to try here is designed to match the patterns which
+  // the GCC driver itself uses, as this is part of the GCC-compatible driver.
+  // This was determined by running GCC in a fake filesystem, creating all
+  // possible permutations of these directories, and seeing which ones it added
+  // to the link paths.
   path_list  = getFilePaths();
 
   const std::string OSLibDir = std::string(getOSLibDir(Triple, Args));
@@ -73,8 +85,11 @@ Hurd::Hurd(const Driver , const llvm::Triple , 
const ArgList )
   ExtraOpts.push_back("--build-id");
 #endif
 
-  // If we are currently running Clang inside of the requested system root, add
-  // its parent library paths to those searched.
+  Generic_GCC::AddMultilibPaths(D, SysRoot, OSLibDir, MultiarchTriple, Paths);
+
+  // Similar to the logic for GCC above, if we currently running Clang inside
+  // of the requested system root, add its parent library paths to
+  // those searched.
   // FIXME: It's not clear whether we should use the driver's installed
   // directory ('Dir' below) or the ResourceDir.
   if (StringRef(D.Dir).startswith(SysRoot)) {
@@ -88,8 +103,11 @@ Hurd::Hurd(const Driver , const llvm::Triple , 
const ArgList )
   addPathIfExists(D, SysRoot + "/usr/lib/" + MultiarchTriple, Paths);
   addPathIfExists(D, SysRoot + "/usr/lib/../" + OSLibDir, Paths);
 
-  // If we are currently running Clang inside of the requested system root, add
-  // its parent library path to those searched.
+  Generic_GCC::AddMultiarchPaths(D, SysRoot, OSLibDir, Paths);
+
+  // Similar to the logic for GCC above, if we are currently running Clang
+  // inside of the requested system root, add its parent library path to those
+  // searched.
   // FIXME: It's not clear whether we should use the driver's installed
   // directory ('Dir' below) or the ResourceDir.
   if (StringRef(D.Dir).startswith(SysRoot))
@@ -156,6 +174,9 @@ void Hurd::AddClangSystemIncludeArgs(const ArgList 
,
 
   // Lacking those, try to detect the correct set of system includes for the
   // target triple.
+
+  AddMultilibIncludeArgs(DriverArgs, CC1Args);
+
   if (getTriple().getArch() == llvm::Triple::x86) {
 std::string Path = SysRoot + "/usr/include/i386-gnu";
 if (D.getVFS().exists(Path))

diff  --git a/clang/test/Driver/hurd.c b/clang/test/Driver/hurd.c
index a6ca8ea3379f..7dbbe923557b 100644
--- a/clang/test/Driver/hurd.c
+++ b/clang/test/Driver/hurd.c
@@ -11,7 +11,7 @@
 // CHECK: "-internal-externc-isystem" "[[SYSROOT]]/usr/include"
 // CHECK: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
 // CHECK: "-dynamic-linker" "/lib/ld.so"
-// CHECK: "crtbegin.o"
+// CHECK: "{{.*}}/usr/lib/gcc/i386-gnu/4.6.0{{/|}}crtbegin.o"
 // CHECK: "-L[[SYSROOT]]/lib/i386-gnu"
 // CHECK: "-L[[SYSROOT]]/lib/../lib32"
 // CHECK: "-L[[SYSROOT]]/usr/lib/i386-gnu"
@@ -33,7 +33,7 @@
 // CHECK-STATIC: "-internal-externc-isystem" "[[SYSROOT]]/usr/include"
 // CHECK-STATIC: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
 // CHECK-STATIC: "-static"
-// CHECK-STATIC: "crtbeginT.o"
+// CHECK-STATIC: "{{.*}}/usr/lib/gcc/i386-gnu/4.6.0{{/|}}crtbeginT.o"
 // CHECK-STATIC: "-L[[SYSROOT]]/lib/i386-gnu"
 // CHECK-STATIC: "-L[[SYSROOT]]/lib/../lib32"
 // CHECK-STATIC: "-L[[SYSROOT]]/usr/lib/i386-gnu"
@@ -53,10 +53,21 @@
 // CHECK-SHARED: "-internal-externc-isystem" 

[clang-tools-extra] c6a65bb - clagn-tidy/doc: Add a link to readability-static-accessed-through-instance from readability-convert-member-functions-to-static

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-04-01T17:01:48+02:00
New Revision: c6a65bb93f218dbdec98f51952a309afda5608ea

URL: 
https://github.com/llvm/llvm-project/commit/c6a65bb93f218dbdec98f51952a309afda5608ea
DIFF: 
https://github.com/llvm/llvm-project/commit/c6a65bb93f218dbdec98f51952a309afda5608ea.diff

LOG: clagn-tidy/doc: Add a link to readability-static-accessed-through-instance 
from readability-convert-member-functions-to-static

Added: 


Modified: 

clang-tools-extra/docs/clang-tidy/checks/readability-convert-member-functions-to-static.rst

Removed: 




diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/readability-convert-member-functions-to-static.rst
 
b/clang-tools-extra/docs/clang-tidy/checks/readability-convert-member-functions-to-static.rst
index 891f6be63714..c2f05cf589ea 100644
--- 
a/clang-tools-extra/docs/clang-tidy/checks/readability-convert-member-functions-to-static.rst
+++ 
b/clang-tools-extra/docs/clang-tidy/checks/readability-convert-member-functions-to-static.rst
@@ -10,5 +10,5 @@ After applying modifications as suggested by the check, 
runnnig the check again
 might find more opportunities to mark member functions ``static``.
 
 After making a member function ``static``, you might want to run the check
-`readability-static-accessed-through-instance` to replace calls like
+`readability-static-accessed-through-instance 
`_ to replace calls like
 ``Instance.method()`` by ``Class::method()``.



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] 72fd103 - Doc: Links should use https

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-03-22T22:49:33+01:00
New Revision: 72fd1033ea577a769cc855fde6b5576b82380715

URL: 
https://github.com/llvm/llvm-project/commit/72fd1033ea577a769cc855fde6b5576b82380715
DIFF: 
https://github.com/llvm/llvm-project/commit/72fd1033ea577a769cc855fde6b5576b82380715.diff

LOG: Doc: Links should use https

Added: 


Modified: 
clang/docs/LibASTImporter.rst
clang/docs/analyzer/checkers.rst
libcxx/docs/TestingLibcxx.rst
libcxx/docs/UsingLibcxx.rst
libcxx/docs/index.rst
libunwind/docs/BuildingLibunwind.rst
libunwind/docs/index.rst
lld/docs/AtomLLD.rst
lld/docs/NewLLD.rst
lld/docs/design.rst
lld/docs/development.rst
lld/docs/getting_started.rst
lld/docs/index.rst
llvm/docs/AMDGPUUsage.rst
llvm/docs/AliasAnalysis.rst
llvm/docs/CMake.rst
llvm/docs/CommandGuide/llvm-lipo.rst
llvm/docs/CommandGuide/llvm-objcopy.rst
llvm/docs/CommandGuide/llvm-objdump.rst
llvm/docs/CommandGuide/llvm-size.rst
llvm/docs/CommandGuide/llvm-strings.rst
llvm/docs/CommandGuide/llvm-strip.rst
llvm/docs/CompileCudaWithLLVM.rst
llvm/docs/FAQ.rst
llvm/docs/GettingStarted.rst
llvm/docs/GettingStartedVS.rst
llvm/docs/GlobalISel/GMIR.rst
llvm/docs/GlobalISel/IRTranslator.rst
llvm/docs/GlobalISel/KnownBits.rst
llvm/docs/HistoricalNotes/2007-OriginalClangReadme.txt
llvm/docs/HowToCrossCompileLLVM.rst
llvm/docs/HowToSetUpLLVMStyleRTTI.rst
llvm/docs/LLVMBuild.txt
llvm/docs/LangRef.rst
llvm/docs/Lexicon.rst
llvm/docs/LibFuzzer.rst
llvm/docs/LoopTerminology.rst
llvm/docs/MarkdownQuickstartTemplate.md
llvm/docs/MergeFunctions.rst
llvm/docs/ProgrammersManual.rst
llvm/docs/Proposals/GitHubMove.rst
llvm/docs/README.txt
llvm/docs/Reference.rst
llvm/docs/ReleaseProcess.rst
llvm/docs/SphinxQuickstartTemplate.rst
llvm/docs/TableGen/index.rst
llvm/docs/TestSuiteGuide.md
llvm/docs/TestingGuide.rst
llvm/docs/TypeMetadata.rst
llvm/docs/UserGuides.rst
llvm/docs/Vectorizers.rst
llvm/docs/WritingAnLLVMPass.rst
llvm/docs/index.rst
llvm/docs/tutorial/BuildingAJIT1.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl02.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl03.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl04.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl05.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl08.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl09.rst
llvm/docs/tutorial/OCamlLangImpl3.rst
llvm/docs/tutorial/OCamlLangImpl5.rst
llvm/docs/tutorial/index.rst
openmp/README.rst
polly/docs/TipsAndTricks.rst

Removed: 




diff  --git a/clang/docs/LibASTImporter.rst b/clang/docs/LibASTImporter.rst
index 9c02b6ae76e9..bedaf527f5e9 100644
--- a/clang/docs/LibASTImporter.rst
+++ b/clang/docs/LibASTImporter.rst
@@ -119,7 +119,7 @@ Now we create the Importer and do the import:
   llvm::Expected ImportedOrErr = Importer.Import(From);
 
 The ``Import`` call returns with ``llvm::Expected``, so, we must check for any 
error.
-Please refer to the `error handling 
`_ 
documentation for details.
+Please refer to the `error handling 
`_ 
documentation for details.
 
 .. code-block:: cpp
 

diff  --git a/clang/docs/analyzer/checkers.rst 
b/clang/docs/analyzer/checkers.rst
index bac1181cced1..0bfb6456dc82 100644
--- a/clang/docs/analyzer/checkers.rst
+++ b/clang/docs/analyzer/checkers.rst
@@ -2199,9 +2199,9 @@ lck_rw_try_lock_exclusive, lck_rw_try_lock_shared, 
pthread_mutex_unlock, pthread
 alpha.unix.SimpleStream (C)
 """
 Check for misuses of stream APIs. Check for misuses of stream APIs: ``fopen, 
fclose``
-(demo checker, the subject of the demo (`Slides 
`_ ,
+(demo checker, the subject of the demo (`Slides 
`_ ,
 `Video `_) by Anna Zaks and Jordan Rose 
presented at the
-`2012 LLVM Developers' Meeting `_).
+`2012 LLVM Developers' Meeting `_).
 
 .. code-block:: c
 

diff  --git a/libcxx/docs/TestingLibcxx.rst b/libcxx/docs/TestingLibcxx.rst
index eaba214390da..d295d13d26f3 100644
--- a/libcxx/docs/TestingLibcxx.rst
+++ b/libcxx/docs/TestingLibcxx.rst
@@ -19,7 +19,7 @@ test libc++.
 
 Please see the `Lit Command Guide`_ for more information about LIT.
 
-.. _LIT Command Guide: http://llvm.org/docs/CommandGuide/lit.html
+.. _LIT Command Guide: https://llvm.org/docs/CommandGuide/lit.html
 
 Setting up the Environment
 --

diff  --git a/libcxx/docs/UsingLibcxx.rst 

[clang] 72fd103 - Doc: Links should use https

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-03-22T22:49:33+01:00
New Revision: 72fd1033ea577a769cc855fde6b5576b82380715

URL: 
https://github.com/llvm/llvm-project/commit/72fd1033ea577a769cc855fde6b5576b82380715
DIFF: 
https://github.com/llvm/llvm-project/commit/72fd1033ea577a769cc855fde6b5576b82380715.diff

LOG: Doc: Links should use https

Added: 


Modified: 
clang/docs/LibASTImporter.rst
clang/docs/analyzer/checkers.rst
libcxx/docs/TestingLibcxx.rst
libcxx/docs/UsingLibcxx.rst
libcxx/docs/index.rst
libunwind/docs/BuildingLibunwind.rst
libunwind/docs/index.rst
lld/docs/AtomLLD.rst
lld/docs/NewLLD.rst
lld/docs/design.rst
lld/docs/development.rst
lld/docs/getting_started.rst
lld/docs/index.rst
llvm/docs/AMDGPUUsage.rst
llvm/docs/AliasAnalysis.rst
llvm/docs/CMake.rst
llvm/docs/CommandGuide/llvm-lipo.rst
llvm/docs/CommandGuide/llvm-objcopy.rst
llvm/docs/CommandGuide/llvm-objdump.rst
llvm/docs/CommandGuide/llvm-size.rst
llvm/docs/CommandGuide/llvm-strings.rst
llvm/docs/CommandGuide/llvm-strip.rst
llvm/docs/CompileCudaWithLLVM.rst
llvm/docs/FAQ.rst
llvm/docs/GettingStarted.rst
llvm/docs/GettingStartedVS.rst
llvm/docs/GlobalISel/GMIR.rst
llvm/docs/GlobalISel/IRTranslator.rst
llvm/docs/GlobalISel/KnownBits.rst
llvm/docs/HistoricalNotes/2007-OriginalClangReadme.txt
llvm/docs/HowToCrossCompileLLVM.rst
llvm/docs/HowToSetUpLLVMStyleRTTI.rst
llvm/docs/LLVMBuild.txt
llvm/docs/LangRef.rst
llvm/docs/Lexicon.rst
llvm/docs/LibFuzzer.rst
llvm/docs/LoopTerminology.rst
llvm/docs/MarkdownQuickstartTemplate.md
llvm/docs/MergeFunctions.rst
llvm/docs/ProgrammersManual.rst
llvm/docs/Proposals/GitHubMove.rst
llvm/docs/README.txt
llvm/docs/Reference.rst
llvm/docs/ReleaseProcess.rst
llvm/docs/SphinxQuickstartTemplate.rst
llvm/docs/TableGen/index.rst
llvm/docs/TestSuiteGuide.md
llvm/docs/TestingGuide.rst
llvm/docs/TypeMetadata.rst
llvm/docs/UserGuides.rst
llvm/docs/Vectorizers.rst
llvm/docs/WritingAnLLVMPass.rst
llvm/docs/index.rst
llvm/docs/tutorial/BuildingAJIT1.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl02.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl03.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl04.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl05.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl08.rst
llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl09.rst
llvm/docs/tutorial/OCamlLangImpl3.rst
llvm/docs/tutorial/OCamlLangImpl5.rst
llvm/docs/tutorial/index.rst
openmp/README.rst
polly/docs/TipsAndTricks.rst

Removed: 




diff  --git a/clang/docs/LibASTImporter.rst b/clang/docs/LibASTImporter.rst
index 9c02b6ae76e9..bedaf527f5e9 100644
--- a/clang/docs/LibASTImporter.rst
+++ b/clang/docs/LibASTImporter.rst
@@ -119,7 +119,7 @@ Now we create the Importer and do the import:
   llvm::Expected ImportedOrErr = Importer.Import(From);
 
 The ``Import`` call returns with ``llvm::Expected``, so, we must check for any 
error.
-Please refer to the `error handling 
`_ 
documentation for details.
+Please refer to the `error handling 
`_ 
documentation for details.
 
 .. code-block:: cpp
 

diff  --git a/clang/docs/analyzer/checkers.rst 
b/clang/docs/analyzer/checkers.rst
index bac1181cced1..0bfb6456dc82 100644
--- a/clang/docs/analyzer/checkers.rst
+++ b/clang/docs/analyzer/checkers.rst
@@ -2199,9 +2199,9 @@ lck_rw_try_lock_exclusive, lck_rw_try_lock_shared, 
pthread_mutex_unlock, pthread
 alpha.unix.SimpleStream (C)
 """
 Check for misuses of stream APIs. Check for misuses of stream APIs: ``fopen, 
fclose``
-(demo checker, the subject of the demo (`Slides 
`_ ,
+(demo checker, the subject of the demo (`Slides 
`_ ,
 `Video `_) by Anna Zaks and Jordan Rose 
presented at the
-`2012 LLVM Developers' Meeting `_).
+`2012 LLVM Developers' Meeting `_).
 
 .. code-block:: c
 

diff  --git a/libcxx/docs/TestingLibcxx.rst b/libcxx/docs/TestingLibcxx.rst
index eaba214390da..d295d13d26f3 100644
--- a/libcxx/docs/TestingLibcxx.rst
+++ b/libcxx/docs/TestingLibcxx.rst
@@ -19,7 +19,7 @@ test libc++.
 
 Please see the `Lit Command Guide`_ for more information about LIT.
 
-.. _LIT Command Guide: http://llvm.org/docs/CommandGuide/lit.html
+.. _LIT Command Guide: https://llvm.org/docs/CommandGuide/lit.html
 
 Setting up the Environment
 --

diff  --git a/libcxx/docs/UsingLibcxx.rst 

[clang-tools-extra] 9860517 - doc: use the right url to bugzilla

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-03-22T22:49:40+01:00
New Revision: 986051749cb12992461626e8e37d7c0923816763

URL: 
https://github.com/llvm/llvm-project/commit/986051749cb12992461626e8e37d7c0923816763
DIFF: 
https://github.com/llvm/llvm-project/commit/986051749cb12992461626e8e37d7c0923816763.diff

LOG: doc: use the right url to bugzilla

Added: 


Modified: 
clang-tools-extra/docs/clang-rename.rst
lld/docs/getting_started.rst
llvm/docs/CommandGuide/llvm-lipo.rst
llvm/docs/CommandGuide/llvm-objcopy.rst
llvm/docs/CommandGuide/llvm-objdump.rst
llvm/docs/CommandGuide/llvm-size.rst
llvm/docs/CommandGuide/llvm-strings.rst
llvm/docs/CommandGuide/llvm-strip.rst
llvm/docs/CompileCudaWithLLVM.rst
llvm/docs/LibFuzzer.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-rename.rst 
b/clang-tools-extra/docs/clang-rename.rst
index ef6ed9cc0814..b45ba01c06a2 100644
--- a/clang-tools-extra/docs/clang-rename.rst
+++ b/clang-tools-extra/docs/clang-rename.rst
@@ -16,7 +16,7 @@ functions, variables, arguments, namespaces etc.
 
 The tool is in a very early development stage, so you might encounter bugs and
 crashes. Submitting reports with information about how to reproduce the issue
-to `the LLVM bugtracker `_ will definitely help the
+to `the LLVM bugtracker `_ will definitely help the
 project. If you have any ideas or suggestions, you might want to put a feature
 request there.
 

diff  --git a/lld/docs/getting_started.rst b/lld/docs/getting_started.rst
index 5cf7acdb6a06..506cb24dde84 100644
--- a/lld/docs/getting_started.rst
+++ b/lld/docs/getting_started.rst
@@ -6,7 +6,7 @@ Getting Started: Building and Running lld
 This page gives you the shortest path to checking out and building lld. If you
 run into problems, please file bugs in the `LLVM Bugzilla`__
 
-__ https://llvm.org/bugs/
+__ https://bugs.llvm.org/
 
 Building lld
 

diff  --git a/llvm/docs/CommandGuide/llvm-lipo.rst 
b/llvm/docs/CommandGuide/llvm-lipo.rst
index 666b48116e1a..20b2984fc9b2 100644
--- a/llvm/docs/CommandGuide/llvm-lipo.rst
+++ b/llvm/docs/CommandGuide/llvm-lipo.rst
@@ -70,4 +70,4 @@ COMMANDS
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .

diff  --git a/llvm/docs/CommandGuide/llvm-objcopy.rst 
b/llvm/docs/CommandGuide/llvm-objcopy.rst
index 468d9ae73fb0..14e08d7641d5 100644
--- a/llvm/docs/CommandGuide/llvm-objcopy.rst
+++ b/llvm/docs/CommandGuide/llvm-objcopy.rst
@@ -545,7 +545,7 @@ Otherwise, it exits with code 0.
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .
 
 There is a known issue with :option:`--input-target` and :option:`--target`
 causing only ``binary`` and ``ihex`` formats to have any effect. Other values

diff  --git a/llvm/docs/CommandGuide/llvm-objdump.rst 
b/llvm/docs/CommandGuide/llvm-objdump.rst
index d3ae1c2b7090..04c76491101d 100644
--- a/llvm/docs/CommandGuide/llvm-objdump.rst
+++ b/llvm/docs/CommandGuide/llvm-objdump.rst
@@ -324,7 +324,7 @@ MACH-O ONLY OPTIONS AND COMMANDS
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .
 
 SEE ALSO
 

diff  --git a/llvm/docs/CommandGuide/llvm-size.rst 
b/llvm/docs/CommandGuide/llvm-size.rst
index 856d2f9ac972..b229bc63d40a 100644
--- a/llvm/docs/CommandGuide/llvm-size.rst
+++ b/llvm/docs/CommandGuide/llvm-size.rst
@@ -195,4 +195,4 @@ Otherwise, it exits with code 0.
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .

diff  --git a/llvm/docs/CommandGuide/llvm-strings.rst 
b/llvm/docs/CommandGuide/llvm-strings.rst
index a393dabd8476..e2fe4cb88c9a 100644
--- a/llvm/docs/CommandGuide/llvm-strings.rst
+++ b/llvm/docs/CommandGuide/llvm-strings.rst
@@ -127,4 +127,4 @@ Otherwise, it exits with code 0.
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .

diff  --git a/llvm/docs/CommandGuide/llvm-strip.rst 
b/llvm/docs/CommandGuide/llvm-strip.rst
index b962a1f96e8a..455dc07e9c5c 100644
--- a/llvm/docs/CommandGuide/llvm-strip.rst
+++ b/llvm/docs/CommandGuide/llvm-strip.rst
@@ -190,7 +190,7 @@ Otherwise, it exits with code 0.
 BUGS
 
 
-To report bugs, please visit .
+To report bugs, please visit .
 
 SEE ALSO
 

diff  --git a/llvm/docs/CompileCudaWithLLVM.rst 
b/llvm/docs/CompileCudaWithLLVM.rst
index c471c48c2d97..a2d7fd0b7453 100644
--- a/llvm/docs/CompileCudaWithLLVM.rst
+++ b/llvm/docs/CompileCudaWithLLVM.rst
@@ -30,7 +30,7 @@ Before you build CUDA code, you'll need to have installed the 
CUDA SDK.  See
 `NVIDIA's CUDA installation guide
 

[clang] 7cfd5de - clang/release notes: s/Subversion/git/

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-03-22T22:18:40+01:00
New Revision: 7cfd5de61b8191bbd5a405a05163b71802e8fb8d

URL: 
https://github.com/llvm/llvm-project/commit/7cfd5de61b8191bbd5a405a05163b71802e8fb8d
DIFF: 
https://github.com/llvm/llvm-project/commit/7cfd5de61b8191bbd5a405a05163b71802e8fb8d.diff

LOG: clang/release notes: s/Subversion/git/

Added: 


Modified: 
clang/docs/ReleaseNotes.rst

Removed: 




diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index 710f005985da..9f20c271b50e 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -304,7 +304,7 @@ Additional Information
 
 A wide variety of additional information is available on the `Clang web
 page `_. The web page contains versions of the
-API documentation which are up-to-date with the Subversion version of
+API documentation which are up-to-date with the Git version of
 the source code. You can access versions of these documents specific to
 this release by going into the "``clang/docs/``" directory in the Clang
 tree.



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 7ae1347 - clang analyzer: Fix the webpage rendering

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-02-17T10:03:20+01:00
New Revision: 7ae1347fb259c6f5feb92567192c0b1032553c02

URL: 
https://github.com/llvm/llvm-project/commit/7ae1347fb259c6f5feb92567192c0b1032553c02
DIFF: 
https://github.com/llvm/llvm-project/commit/7ae1347fb259c6f5feb92567192c0b1032553c02.diff

LOG: clang analyzer: Fix the webpage rendering

Added: 


Modified: 
clang/www/analyzer/alpha_checks.html

Removed: 




diff  --git a/clang/www/analyzer/alpha_checks.html 
b/clang/www/analyzer/alpha_checks.html
index 3d4075e5aaf9..181ce1b9de59 100644
--- a/clang/www/analyzer/alpha_checks.html
+++ b/clang/www/analyzer/alpha_checks.html
@@ -1083,9 +1083,6 @@ Non-determinism 
Alpha Checkers
f(i);
 }
 
-
-
-
 
 alpha.nondeterminism.PointerSorting
 (C++)



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 68cd283 - clang-tidy doc: unbreak the CI

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-10T22:25:01+01:00
New Revision: 68cd283f3b074e3b64b9f65e93ceb2de6807c72d

URL: 
https://github.com/llvm/llvm-project/commit/68cd283f3b074e3b64b9f65e93ceb2de6807c72d
DIFF: 
https://github.com/llvm/llvm-project/commit/68cd283f3b074e3b64b9f65e93ceb2de6807c72d.diff

LOG: clang-tidy doc: unbreak the CI

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 3cdf2c9a7d14..c89ec0c10917 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -7,6 +7,8 @@ Clang-Tidy Checks
:glob:
:hidden:
 
+   *
+
 .. csv-table::
:header: "Name", "Offers fixes"
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] faeeb71 - clang-tidy doc: Refresh the list of checkers and polish the script

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-10T22:07:47+01:00
New Revision: faeeb71a17344171f814144213ac4fbc93be28fd

URL: 
https://github.com/llvm/llvm-project/commit/faeeb71a17344171f814144213ac4fbc93be28fd
DIFF: 
https://github.com/llvm/llvm-project/commit/faeeb71a17344171f814144213ac4fbc93be28fd.diff

LOG: clang-tidy doc: Refresh the list of checkers and polish the script

Added: 


Modified: 
clang-tools-extra/clang-tidy/add_new_check.py
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/add_new_check.py 
b/clang-tools-extra/clang-tidy/add_new_check.py
index 68a723900751..4406efc1b0f5 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -368,14 +368,12 @@ def format_link_alias(doc_file):
   f.write(line)
   if line.strip() == ".. csv-table::":
 # We dump the checkers
-f.write('   :header: "Name", "Offers fixes"\n')
-f.write('   :widths: 50, 20\n\n')
+f.write('   :header: "Name", "Offers fixes"\n\n')
 f.writelines(checks)
 # and the aliases
 f.write('\n\n')
 f.write('.. csv-table:: Aliases..\n')
-f.write('   :header: "Name", "Redirect", "Offers fixes"\n')
-f.write('   :widths: 50, 50, 10\n\n')
+f.write('   :header: "Name", "Redirect", "Offers fixes"\n\n')
 f.writelines(checks_alias)
 break
 

diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index e5cac0312081..3cdf2c9a7d14 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -3,6 +3,10 @@
 Clang-Tidy Checks
 =
 
+.. toctree::
+   :glob:
+   :hidden:
+
 .. csv-table::
:header: "Name", "Offers fixes"
 
@@ -25,8 +29,8 @@ Clang-Tidy Checks
`abseil-upgrade-duration-conversions 
`_, "Yes"
`android-cloexec-accept `_, "Yes"
`android-cloexec-accept4 `_,
-   `android-cloexec-creat `_,
-   `android-cloexec-dup `_,
+   `android-cloexec-creat `_, "Yes"
+   `android-cloexec-dup `_, "Yes"
`android-cloexec-epoll-create `_,
`android-cloexec-epoll-create1 `_,
`android-cloexec-fopen `_,
@@ -34,10 +38,10 @@ Clang-Tidy Checks
`android-cloexec-inotify-init1 `_,
`android-cloexec-memfd-create `_,
`android-cloexec-open `_,
-   `android-cloexec-pipe `_,
+   `android-cloexec-pipe `_, "Yes"
`android-cloexec-pipe2 `_,
`android-cloexec-socket `_,
-   `android-comparison-in-temp-failure-retry 
`_, "Yes"
+   `android-comparison-in-temp-failure-retry 
`_,
`boost-use-to-string `_, "Yes"
`bugprone-argument-comment `_, "Yes"
`bugprone-assert-side-effect `_,
@@ -58,8 +62,8 @@ Clang-Tidy Checks
`bugprone-lambda-function-name `_,
`bugprone-macro-parentheses `_, "Yes"
`bugprone-macro-repeated-side-effects 
`_,
-   `bugprone-misplaced-operator-in-strlen-in-alloc 
`_,
-   `bugprone-misplaced-widening-cast 
`_, "Yes"
+   `bugprone-misplaced-operator-in-strlen-in-alloc 
`_, "Yes"
+   `bugprone-misplaced-widening-cast `_,
`bugprone-move-forwarding-reference 
`_, "Yes"
`bugprone-multiple-statement-macro 
`_,
`bugprone-not-null-terminated-result 
`_, "Yes"
@@ -149,19 +153,19 @@ Clang-Tidy Checks
`fuchsia-statically-constructed-objects 
`_,
`fuchsia-trailing-return `_,
`fuchsia-virtual-inheritance `_,
-   `google-build-explicit-make-pair `_, 
"Yes"
+   `google-build-explicit-make-pair `_,
`google-build-namespaces `_,
`google-build-using-namespace `_,
`google-default-arguments `_,
`google-explicit-constructor `_, "Yes"
-   `google-global-names-in-headers `_, 
"Yes"
-   `google-objc-avoid-nsobject-new `_, 
"Yes"
+   `google-global-names-in-headers `_,
+   `google-objc-avoid-nsobject-new `_,
`google-objc-avoid-throwing-exception 
`_,
-   `google-objc-function-naming `_, "Yes"
+   `google-objc-function-naming `_,
`google-objc-global-variable-declaration 
`_,
`google-readability-avoid-underscore-in-googletest-name 
`_,
-   `google-readability-casting `_, "Yes"
-   `google-readability-todo `_, "Yes"
+   `google-readability-casting `_,
+   `google-readability-todo `_,
`google-runtime-int `_,
`google-runtime-operator `_,
`google-runtime-references `_,
@@ -197,7 +201,7 @@ Clang-Tidy Checks
`modernize-deprecated-headers `_, "Yes"
`modernize-deprecated-ios-base-aliases 
`_, "Yes"
`modernize-loop-convert `_, "Yes"
-   `modernize-make-shared `_, "Yes"
+   `modernize-make-shared `_,
`modernize-make-unique `_, "Yes"
`modernize-pass-by-value `_, "Yes"
`modernize-raw-string-literal `_, "Yes"
@@ -237,19 +241,19 @@ Clang-Tidy Checks
`performance-inefficient-string-concatenation 
`_,
`performance-inefficient-vector-operation 
`_, "Yes"
`performance-move-const-arg `_, "Yes"
-   

[clang-tools-extra] f3db1aa - MakeUniqueCheck.cpp: explicit the fact that there is an autofix for this checker

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-10T22:06:03+01:00
New Revision: f3db1aad2796c62f0d188a74f2901c18e51843c2

URL: 
https://github.com/llvm/llvm-project/commit/f3db1aad2796c62f0d188a74f2901c18e51843c2
DIFF: 
https://github.com/llvm/llvm-project/commit/f3db1aad2796c62f0d188a74f2901c18e51843c2.diff

LOG: MakeUniqueCheck.cpp: explicit the fact that there is an autofix for this 
checker

Added: 


Modified: 
clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp 
b/clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp
index 1ee4fd701d8c..e2f812b35aea 100644
--- a/clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp
+++ b/clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp
@@ -41,6 +41,8 @@ bool MakeUniqueCheck::isLanguageVersionSupported(
   return RequireCPlusPlus14 ? LangOpts.CPlusPlus14 : LangOpts.CPlusPlus11;
 }
 
+// FixItHint is done by MakeSmartPtrCheck
+
 } // namespace modernize
 } // namespace tidy
 } // namespace clang



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] b38d0d5 - clang-tidy doc - remove the widths

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-10T08:11:05+01:00
New Revision: b38d0d5bdb353f8496a0fc38b9bbee419b41a321

URL: 
https://github.com/llvm/llvm-project/commit/b38d0d5bdb353f8496a0fc38b9bbee419b41a321
DIFF: 
https://github.com/llvm/llvm-project/commit/b38d0d5bdb353f8496a0fc38b9bbee419b41a321.diff

LOG: clang-tidy doc - remove the widths

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 1841588b7af3..e5cac0312081 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -5,7 +5,6 @@ Clang-Tidy Checks
 
 .. csv-table::
:header: "Name", "Offers fixes"
-   :widths: 50, 20
 
`abseil-duration-addition `_, "Yes"
`abseil-duration-comparison `_, "Yes"
@@ -287,7 +286,6 @@ Clang-Tidy Checks
 
 .. csv-table:: Aliases..
:header: "Name", "Redirect", "Offers fixes"
-   :widths: 50, 50, 10
 
`cert-dcl03-c `_, `misc-static-assert 
`_, "Yes"
`cert-dcl16-c `_, `readability-uppercase-literal-suffix 
`_, "Yes"



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] c348a26 - [clang-tidy] Refresh the add_new_check.py now that we use a table + autofix

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-09T22:11:18+01:00
New Revision: c348a2674b5753afde6842d5a6ee75db67e3

URL: 
https://github.com/llvm/llvm-project/commit/c348a2674b5753afde6842d5a6ee75db67e3
DIFF: 
https://github.com/llvm/llvm-project/commit/c348a2674b5753afde6842d5a6ee75db67e3.diff

LOG: [clang-tidy] Refresh the add_new_check.py now that we use a table + autofix

Reviewers: alexfh

Reviewed By: alexfh

Subscribers: njames93, xazax.hun, mgorny, cfe-commits

Tags: #clang, #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D72421

Added: 


Modified: 
clang-tools-extra/clang-tidy/add_new_check.py

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/add_new_check.py 
b/clang-tools-extra/clang-tidy/add_new_check.py
index 0a8a11b8d0dd..68a723900751 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -294,37 +294,89 @@ def write_test(module_path, module, check_name, 
test_extension):
 def update_checks_list(clang_tidy_path):
   docs_dir = os.path.join(clang_tidy_path, '../docs/clang-tidy/checks')
   filename = os.path.normpath(os.path.join(docs_dir, 'list.rst'))
+  # Read the content of the current list.rst file
   with open(filename, 'r') as f:
 lines = f.readlines()
+  # Get all existing docs
   doc_files = list(filter(lambda s: s.endswith('.rst') and s != 'list.rst',
  os.listdir(docs_dir)))
   doc_files.sort()
 
-  def format_link(doc_file):
+  def has_auto_fix(check_name):
+dirname, _, check_name = check_name.partition("-")
+
+checkerCode = os.path.join(dirname, get_camel_name(check_name)) + ".cpp"
+
+if not os.path.isfile(checkerCode):
+  return ""
+
+with open(checkerCode) as f:
+  code = f.read()
+  if 'FixItHint' in code or "ReplacementText" in code or "fixit" in code:
+# Some simple heuristics to figure out if a checker has an autofix or 
not.
+return ' "Yes"'
+return ""
+
+  def process_doc(doc_file):
 check_name = doc_file.replace('.rst', '')
+
 with open(os.path.join(docs_dir, doc_file), 'r') as doc:
   content = doc.read()
   match = re.search('.*:orphan:.*', content)
+
   if match:
-return ''
+# Orphan page, don't list it.
+return '', ''
 
   match = re.search('.*:http-equiv=refresh: \d+;URL=(.*).html.*',
 content)
-  if match:
-return '   %(check)s (redirects to %(target)s) <%(check)s>\n' % {
-'check': check_name,
-'target': match.group(1)
-}
-  return '   %s\n' % check_name
+  # Is it a redirect?
+  return check_name, match
+
+  def format_link(doc_file):
+check_name, match = process_doc(doc_file)
+if not match and check_name:
+  return '   `%(check)s <%(check)s.html>`_,%(autofix)s\n' % {
+'check': check_name,
+'autofix': has_auto_fix(check_name)
+  }
+else:
+  return ''
+
+  def format_link_alias(doc_file):
+check_name, match = process_doc(doc_file)
+if match and check_name:
+  if match.group(1) == 'https://clang.llvm.org/docs/analyzer/checkers':
+title_redirect = 'Clang Static Analyzer'
+  else:
+title_redirect = match.group(1)
+  # The checker is just a redirect.
+  return '   `%(check)s <%(check)s.html>`_, `%(title)s 
<%(target)s.html>`_,%(autofix)s\n' % {
+'check': check_name,
+'target': match.group(1),
+'title': title_redirect,
+'autofix': has_auto_fix(match.group(1))
+  }
+return ''
 
   checks = map(format_link, doc_files)
+  checks_alias = map(format_link_alias, doc_files)
 
   print('Updating %s...' % filename)
   with open(filename, 'w') as f:
 for line in lines:
   f.write(line)
-  if line.startswith('.. toctree::'):
+  if line.strip() == ".. csv-table::":
+# We dump the checkers
+f.write('   :header: "Name", "Offers fixes"\n')
+f.write('   :widths: 50, 20\n\n')
 f.writelines(checks)
+# and the aliases
+f.write('\n\n')
+f.write('.. csv-table:: Aliases..\n')
+f.write('   :header: "Name", "Redirect", "Offers fixes"\n')
+f.write('   :widths: 50, 50, 10\n\n')
+f.writelines(checks_alias)
 break
 
 
@@ -345,6 +397,11 @@ def write_docs(module_path, module, check_name):
'underline': '=' * len(check_name_dashes)})
 
 
+def get_camel_name(check_name):
+  return ''.join(map(lambda elem: elem.capitalize(),
+ check_name.split('-'))) + 'Check'
+
+
 def main():
   language_to_extension = {
   'c': 'c',
@@ -384,13 +441,11 @@ def main():
 
   module = args.module
   check_name = args.check
-
+  check_name_camel = get_camel_name(check_name)
   if check_name.startswith(module):
 print('Check name "%s" must not start with the module "%s". Exiting.' % (
 

Re: [clang-tools-extra] d2c9c91 - Move from a long list of checkers to tables

[Sylvestre Ledru via cfe-commits]

Le 08/01/2020 à 14:14, Alexander Kornienko a écrit :

[adding the list back]

This list is one of the places that would be quite boring and error-prone to update manually. I'd strongly prefer to have an automatic way of updating it. Detecting whether a check has a fix might be not an easy task though. Maybe the script could use some simple heuristic to detect obvious ways 
to add a fix (e.g. search for /FixItHint::Create.*\(|fixit::create.*\(/ - maybe in non-comment lines, if it turns out to be feasible - in the check's source files) and mark these checks as having fixes, the ones that don't have the word "fix" most probably don't generate a fix, the rest are 
"unknown" (and these should be fixed by hand). WDYT?


This is basically what I have done (greping the code to see if there is an 
autofix).

But I missed modernize-make-unique as it doesn't contain any reference to fixit 
:/

(I could just update modernize/MakeUniqueCheck.cpp to add a comment)

Cheers,

Sylvestre


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 773667b - clang-tidy doc: Remove severities as they don't make consensus

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2020-01-01T15:42:46+01:00
New Revision: 773667b8c20d35c18334f8c7663df8ceacfdd2e5

URL: 
https://github.com/llvm/llvm-project/commit/773667b8c20d35c18334f8c7663df8ceacfdd2e5
DIFF: 
https://github.com/llvm/llvm-project/commit/773667b8c20d35c18334f8c7663df8ceacfdd2e5.diff

LOG: clang-tidy doc: Remove severities as they don't make consensus

Reviewers: jdoerfert, aaron.ballman

Reviewed By: aaron.ballman

Subscribers: whisperity, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D72049

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 7273f2f2c10a..39eca157244f 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -3,405 +3,401 @@
 Clang-Tidy Checks
 =
 
-.. Severities are coming from Codechecker:
-   
https://github.com/Ericsson/codechecker/blob/master/config/checker_severity_map.json
-   If you change it here, please submit a PR on codechecker too
-
 .. csv-table::
-   :header: "Name", "Offers fixes", "Severity"
-   :widths: 50, 20, 10
+   :header: "Name", "Offers fixes"
+   :widths: 50, 20
 
-   `abseil-duration-addition `_, "Yes", ""
-   `abseil-duration-comparison `_, "Yes", ""
-   `abseil-duration-conversion-cast `_, 
"Yes", ""
-   `abseil-duration-division `_, "Yes", ""
-   `abseil-duration-factory-float `_, 
"Yes", ""
-   `abseil-duration-factory-scale `_, 
"Yes", ""
-   `abseil-duration-subtraction `_, "Yes", ""
-   `abseil-duration-unnecessary-conversion 
`_, "Yes", ""
-   `abseil-faster-strsplit-delimiter 
`_, "Yes", ""
-   `abseil-no-internal-dependencies `_, 
, ""
-   `abseil-no-namespace `_, , ""
-   `abseil-redundant-strcat-calls `_, 
"Yes", ""
-   `abseil-str-cat-append `_, "Yes", ""
-   `abseil-string-find-startswith `_, 
"Yes", "style"
-   `abseil-time-comparison `_, "Yes", ""
-   `abseil-time-subtraction `_, "Yes", ""
-   `abseil-upgrade-duration-conversions 
`_, "Yes", ""
-   `android-cloexec-accept `_, "Yes", ""
-   `android-cloexec-accept4 `_, , ""
-   `android-cloexec-creat `_, , "medium"
-   `android-cloexec-dup `_, , ""
-   `android-cloexec-epoll-create `_, , ""
-   `android-cloexec-epoll-create1 `_, , ""
-   `android-cloexec-fopen `_, , "medium"
-   `android-cloexec-inotify-init `_, , ""
-   `android-cloexec-inotify-init1 `_, , ""
-   `android-cloexec-memfd-create `_, , ""
-   `android-cloexec-open `_, , "medium"
-   `android-cloexec-pipe `_, , ""
-   `android-cloexec-pipe2 `_, , ""
-   `android-cloexec-socket `_, , "medium"
-   `android-comparison-in-temp-failure-retry 
`_, "Yes", ""
-   `boost-use-to-string `_, "Yes", "low"
-   `bugprone-argument-comment `_, "Yes", "low"
-   `bugprone-assert-side-effect `_, , 
"medium"
-   `bugprone-bad-signal-to-kill-thread 
`_, , ""
-   `bugprone-bool-pointer-implicit-conversion 
`_, "Yes", "low"
-   `bugprone-branch-clone `_, , "low"
-   `bugprone-copy-constructor-init `_, 
"Yes", "medium"
-   `bugprone-dangling-handle `_, , "high"
-   `bugprone-dynamic-static-initializers 
`_, , ""
-   `bugprone-exception-escape `_, , "medium"
-   `bugprone-fold-init-type `_, , "high"
-   `bugprone-forward-declaration-namespace 
`_, , "low"
-   `bugprone-forwarding-reference-overload 
`_, , "low"
-   `bugprone-inaccurate-erase `_, "Yes", "high"
-   `bugprone-incorrect-roundings `_, , 
"high"
-   `bugprone-infinite-loop `_, , "medium"
-   `bugprone-integer-division `_, , "medium"
-   `bugprone-lambda-function-name `_, , 
"low"
-   `bugprone-macro-parentheses `_, "Yes", 
"medium"
-   `bugprone-macro-repeated-side-effects 
`_, , "medium"
-   `bugprone-misplaced-operator-in-strlen-in-alloc 
`_, , "medium"
-   `bugprone-misplaced-widening-cast 
`_, "Yes", "high"
-   `bugprone-move-forwarding-reference 
`_, "Yes", "medium"
-   `bugprone-multiple-statement-macro 
`_, , "medium"
-   `bugprone-not-null-terminated-result 
`_, "Yes", "medium"
-   `bugprone-parent-virtual-call `_, "Yes", 
"medium"
-   `bugprone-posix-return `_, "Yes", ""
-   `bugprone-sizeof-container `_, , "high"
-   `bugprone-sizeof-expression `_, , "high"
-   `bugprone-string-constructor `_, "Yes", 
"high"
-   `bugprone-string-integer-assignment 
`_, "Yes", "low"
-   `bugprone-string-literal-with-embedded-nul 
`_, , "medium"
-   `bugprone-suspicious-enum-usage `_, , 
"high"
-   `bugprone-suspicious-memset-usage 
`_, "Yes", "high"
-   `bugprone-suspicious-missing-comma 
`_, , "high"
-   `bugprone-suspicious-semicolon `_, 
"Yes", "high"
-   `bugprone-suspicious-string-compare 
`_, "Yes", "medium"
-   `bugprone-swapped-arguments `_, "Yes", 
"high"
-   `bugprone-terminating-continue `_, 
"Yes", "medium"
-   `bugprone-throw-keyword-missing `_, , 
"medium"
-   `bugprone-too-small-loop-variable 
`_, , "medium"
-   

[clang-tools-extra] e8c9110 - clang-tidy doc: modernize-make-unique has an autofix

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-31T11:56:17+01:00
New Revision: e8c9110b56b516a22b41e95e347bc141814ab87c

URL: 
https://github.com/llvm/llvm-project/commit/e8c9110b56b516a22b41e95e347bc141814ab87c
DIFF: 
https://github.com/llvm/llvm-project/commit/e8c9110b56b516a22b41e95e347bc141814ab87c.diff

LOG: clang-tidy doc: modernize-make-unique has an autofix

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 08805ea248eb..7273f2f2c10a 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -202,7 +202,7 @@ Clang-Tidy Checks
`modernize-deprecated-ios-base-aliases 
`_, "Yes", "low"
`modernize-loop-convert `_, "Yes", "style"
`modernize-make-shared `_, "Yes", "low"
-   `modernize-make-unique `_, , "low"
+   `modernize-make-unique `_, "Yes", "low"
`modernize-pass-by-value `_, "Yes", "low"
`modernize-raw-string-literal `_, "Yes", 
"style"
`modernize-redundant-void-arg `_, "Yes", 
"style"



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] c96c606 - clang-doc remove trailing whitespaces

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-24T13:07:08+01:00
New Revision: c96c606b85c6a1e046fb8b1fb5bd1cbf28d80169

URL: 
https://github.com/llvm/llvm-project/commit/c96c606b85c6a1e046fb8b1fb5bd1cbf28d80169
DIFF: 
https://github.com/llvm/llvm-project/commit/c96c606b85c6a1e046fb8b1fb5bd1cbf28d80169.diff

LOG: clang-doc remove trailing whitespaces

Added: 


Modified: 
clang-tools-extra/docs/clang-doc.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-doc.rst 
b/clang-tools-extra/docs/clang-doc.rst
index 93a6f67e2c8f..328416ee6a38 100644
--- a/clang-tools-extra/docs/clang-doc.rst
+++ b/clang-tools-extra/docs/clang-doc.rst
@@ -7,8 +7,8 @@ Clang-Doc
 .. toctree::
:maxdepth: 1
 
-:program:`clang-doc` is a tool for generating C and C++ documentation from 
-source code and comments. 
+:program:`clang-doc` is a tool for generating C and C++ documentation from
+source code and comments.
 
 The tool is in a very early development stage, so you might encounter bugs and
 crashes. Submitting reports with information about how to reproduce the issue
@@ -21,7 +21,7 @@ Use
 
 :program:`clang-doc` is a `LibTooling
 `_-based tool, and so requires a
-compile command database for your project (for an example of how to do this 
+compile command database for your project (for an example of how to do this
 see `How To Setup Tooling For LLVM
 `_).
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 8131c04 - doc: Document that extra-arg/extra-arg-before can be used several times

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-24T13:07:08+01:00
New Revision: 8131c04836829e7f37f2feaa1d85b4ef62ad092f

URL: 
https://github.com/llvm/llvm-project/commit/8131c04836829e7f37f2feaa1d85b4ef62ad092f
DIFF: 
https://github.com/llvm/llvm-project/commit/8131c04836829e7f37f2feaa1d85b4ef62ad092f.diff

LOG: doc: Document that extra-arg/extra-arg-before can be used several times

Hopefully, it will help other people

Added: 


Modified: 
clang-tools-extra/docs/clang-doc.rst
clang-tools-extra/docs/clang-rename.rst
clang-tools-extra/docs/clang-tidy/index.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-doc.rst 
b/clang-tools-extra/docs/clang-doc.rst
index 328416ee6a38..9be8a8dc31d4 100644
--- a/clang-tools-extra/docs/clang-doc.rst
+++ b/clang-tools-extra/docs/clang-doc.rst
@@ -81,7 +81,9 @@ Options
 
 --doxygen   - Use only doxygen-style comments to generate 
docs.
 --extra-arg=- Additional argument to append to the 
compiler command line
+  Can be used several times.
 --extra-arg-before= - Additional argument to prepend to the 
compiler command line
+  Can be used several times.
 --format=- Format for outputted docs.
   =yaml -   Documentation in YAML format.
   =md   -   Documentation in MD format.

diff  --git a/clang-tools-extra/docs/clang-rename.rst 
b/clang-tools-extra/docs/clang-rename.rst
index 2796141f8545..ef6ed9cc0814 100644
--- a/clang-tools-extra/docs/clang-rename.rst
+++ b/clang-tools-extra/docs/clang-rename.rst
@@ -123,7 +123,9 @@ That way you can avoid spelling out all the names as 
command line arguments:
 
 -export-fixes=   - YAML file to store suggested fixes in.
 -extra-arg=- Additional argument to append to the compiler 
command line
+ Can be used several times.
 -extra-arg-before= - Additional argument to prepend to the 
compiler command line
+ Can be used several times.
 -force - Ignore nonexistent qualified names.
 -i - Overwrite edited s.
 -input=- YAML file to load oldname-newname pairs from.

diff  --git a/clang-tools-extra/docs/clang-tidy/index.rst 
b/clang-tools-extra/docs/clang-tidy/index.rst
index 54011c38d32e..4956def46c52 100644
--- a/clang-tools-extra/docs/clang-tidy/index.rst
+++ b/clang-tools-extra/docs/clang-tidy/index.rst
@@ -155,7 +155,9 @@ An overview of all the command-line options:
  stored fixes can be applied to the input 
source
  code with clang-apply-replacements.
 --extra-arg=   - Additional argument to append to the 
compiler command line
+ Can be used several times.
 --extra-arg-before=- Additional argument to prepend to the 
compiler command line
+ Can be used several times.
 --fix  -
  Apply suggested fixes. Without -fix-errors
  clang-tidy will bail out if any 
compilation



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] d2c9c91 - Move from a long list of checkers to tables

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-23T18:44:31+01:00
New Revision: d2c9c9157b0528436d3b9f5e22c872f0ee6509a2

URL: 
https://github.com/llvm/llvm-project/commit/d2c9c9157b0528436d3b9f5e22c872f0ee6509a2
DIFF: 
https://github.com/llvm/llvm-project/commit/d2c9c9157b0528436d3b9f5e22c872f0ee6509a2.diff

LOG: Move from a long list of checkers to tables

Summary:
Currently, the list isn't very useful.
This change adds two tables.

* The checkers
* The aliases

For each checkers, we provide extract info:

* the severity. Taken from codechecker - 
https://github.com/Ericsson/codechecker/blob/master/config/checker_severity_map.json
* if the checker has an autofix or not

I used the cvs format for the table because:
* it is easy
* the data could be reused by other tools (we could move
that into a separated / generated file at some point)

Reviewers: alexfh, jdoerfert, jfb, lebedev.ri, Eugene.Zelenko

Subscribers: dexonsmith, wuzish, nemanjai, kbarton, arphaman, lebedev.ri, 
whisperity, Eugene.Zelenko, JonasToth, JDevlieghere, xazax.hun, cfe-commits, 
#clang-tools-extra

Tags: #clang

Differential Revision: https://reviews.llvm.org/D36051

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 565fe7a03c09..08805ea248eb 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -3,388 +3,408 @@
 Clang-Tidy Checks
 =
 
+.. Severities are coming from Codechecker:
+   
https://github.com/Ericsson/codechecker/blob/master/config/checker_severity_map.json
+   If you change it here, please submit a PR on codechecker too
+
+.. csv-table::
+   :header: "Name", "Offers fixes", "Severity"
+   :widths: 50, 20, 10
+
+   `abseil-duration-addition `_, "Yes", ""
+   `abseil-duration-comparison `_, "Yes", ""
+   `abseil-duration-conversion-cast `_, 
"Yes", ""
+   `abseil-duration-division `_, "Yes", ""
+   `abseil-duration-factory-float `_, 
"Yes", ""
+   `abseil-duration-factory-scale `_, 
"Yes", ""
+   `abseil-duration-subtraction `_, "Yes", ""
+   `abseil-duration-unnecessary-conversion 
`_, "Yes", ""
+   `abseil-faster-strsplit-delimiter 
`_, "Yes", ""
+   `abseil-no-internal-dependencies `_, 
, ""
+   `abseil-no-namespace `_, , ""
+   `abseil-redundant-strcat-calls `_, 
"Yes", ""
+   `abseil-str-cat-append `_, "Yes", ""
+   `abseil-string-find-startswith `_, 
"Yes", "style"
+   `abseil-time-comparison `_, "Yes", ""
+   `abseil-time-subtraction `_, "Yes", ""
+   `abseil-upgrade-duration-conversions 
`_, "Yes", ""
+   `android-cloexec-accept `_, "Yes", ""
+   `android-cloexec-accept4 `_, , ""
+   `android-cloexec-creat `_, , "medium"
+   `android-cloexec-dup `_, , ""
+   `android-cloexec-epoll-create `_, , ""
+   `android-cloexec-epoll-create1 `_, , ""
+   `android-cloexec-fopen `_, , "medium"
+   `android-cloexec-inotify-init `_, , ""
+   `android-cloexec-inotify-init1 `_, , ""
+   `android-cloexec-memfd-create `_, , ""
+   `android-cloexec-open `_, , "medium"
+   `android-cloexec-pipe `_, , ""
+   `android-cloexec-pipe2 `_, , ""
+   `android-cloexec-socket `_, , "medium"
+   `android-comparison-in-temp-failure-retry 
`_, "Yes", ""
+   `boost-use-to-string `_, "Yes", "low"
+   `bugprone-argument-comment `_, "Yes", "low"
+   `bugprone-assert-side-effect `_, , 
"medium"
+   `bugprone-bad-signal-to-kill-thread 
`_, , ""
+   `bugprone-bool-pointer-implicit-conversion 
`_, "Yes", "low"
+   `bugprone-branch-clone `_, , "low"
+   `bugprone-copy-constructor-init `_, 
"Yes", "medium"
+   `bugprone-dangling-handle `_, , "high"
+   `bugprone-dynamic-static-initializers 
`_, , ""
+   `bugprone-exception-escape `_, , "medium"
+   `bugprone-fold-init-type `_, , "high"
+   `bugprone-forward-declaration-namespace 
`_, , "low"
+   `bugprone-forwarding-reference-overload 
`_, , "low"
+   `bugprone-inaccurate-erase `_, "Yes", "high"
+   `bugprone-incorrect-roundings `_, , 
"high"
+   `bugprone-infinite-loop `_, , "medium"
+   `bugprone-integer-division `_, , "medium"
+   `bugprone-lambda-function-name `_, , 
"low"
+   `bugprone-macro-parentheses `_, "Yes", 
"medium"
+   `bugprone-macro-repeated-side-effects 
`_, , "medium"
+   `bugprone-misplaced-operator-in-strlen-in-alloc 
`_, , "medium"
+   `bugprone-misplaced-widening-cast 
`_, "Yes", "high"
+   `bugprone-move-forwarding-reference 
`_, "Yes", "medium"
+   `bugprone-multiple-statement-macro 
`_, , "medium"
+   `bugprone-not-null-terminated-result 
`_, "Yes", "medium"
+   `bugprone-parent-virtual-call `_, "Yes", 
"medium"
+   `bugprone-posix-return `_, "Yes", ""
+   `bugprone-sizeof-container `_, , "high"
+   `bugprone-sizeof-expression `_, , "high"
+   `bugprone-string-constructor `_, "Yes", 
"high"
+   `bugprone-string-integer-assignment 
`_, "Yes", "low"
+   

[clang] a0ce615 - clang is now under the apache2 license

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-21T10:52:30+01:00
New Revision: a0ce6155734eaaea315dc82579b230b172fb8ccc

URL: 
https://github.com/llvm/llvm-project/commit/a0ce6155734eaaea315dc82579b230b172fb8ccc
DIFF: 
https://github.com/llvm/llvm-project/commit/a0ce6155734eaaea315dc82579b230b172fb8ccc.diff

LOG: clang is now under the apache2 license

Added: 


Modified: 
clang/www/features.html
clang/www/index.html

Removed: 




diff  --git a/clang/www/features.html b/clang/www/features.html
index 715c9d18dcab..d0b4a7f30fd3 100755
--- a/clang/www/features.html
+++ b/clang/www/features.html
@@ -269,19 +269,18 @@ Integration with IDEs
 
 
 
-Use the LLVM 'BSD' License
+Use the LLVM 'Apache 2' License
 
 
 We actively intend for clang (and LLVM as a whole) to be used for
 commercial projects, not only as a stand-alone compiler but also as a library
-embedded inside a proprietary application.  The BSD license is the simplest way
-to allow this.  We feel that the license encourages contributors to pick up the
-source and work with it, and believe that those individuals and organizations
-will contribute back their work if they do not want to have to maintain a fork
-forever (which is time consuming and expensive when merges are involved).
-Further, nobody makes money on compilers these days, but many people need them
-to get bigger goals accomplished: it makes sense for everyone to work
-together.
+embedded inside a proprietary application. We feel that the license encourages
+contributors to pick up the source and work with it, and believe that those
+individuals and organizations will contribute back their work if they do not
+want to have to maintain a fork forever (which is time consuming and expensive
+when merges are involved). Further, nobody makes money on compilers these days,
+but many people need them to get bigger goals accomplished: it makes sense for
+everyone to work together.
 
 For more information about the LLVM/clang license, please see the https://llvm.org/docs/DeveloperPolicy.html#copyright-license-and-patents;>LLVM
 License

diff  --git a/clang/www/index.html b/clang/www/index.html
index ed826b8ebbdf..ce400d6153d3 100755
--- a/clang/www/index.html
+++ b/clang/www/index.html
@@ -44,7 +44,7 @@ Features and Goals
   Support diverse clients (refactoring, static analysis, code generation,
etc.)
   Allow tight integration with IDEs
-  Use the LLVM 'BSD' License
+  Use the LLVM 'Apache 2' License
   
 
   Internal Design and



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 384a287 - Remove a gcc 4.9 comparison as it doesn't make sense

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-21T10:46:45+01:00
New Revision: 384a287a87ad0989624eb785293d38f9b710

URL: 
https://github.com/llvm/llvm-project/commit/384a287a87ad0989624eb785293d38f9b710
DIFF: 
https://github.com/llvm/llvm-project/commit/384a287a87ad0989624eb785293d38f9b710.diff

LOG: Remove a gcc 4.9 comparison as it doesn't make sense

Added: 


Modified: 
clang/www/features.html

Removed: 




diff  --git a/clang/www/features.html b/clang/www/features.html
index aeab4e56969d..715c9d18dcab 100755
--- a/clang/www/features.html
+++ b/clang/www/features.html
@@ -91,15 +91,9 @@ Expressive Diagnostics
 highlighting related information so that it is easy to understand at a glance,
 and making the wording as clear as possible.
 
-Here is one simple example that illustrates the 
diff erence between a typical
-GCC and Clang diagnostic:
+Here is one simple example that illustrates the quality of Clang 
diagnostic:
 
 
-  $ gcc-4.9 -fsyntax-only t.c
-  t.c: In function 'int f(int, int)':
-  t.c:7:39: error: invalid operands to binary + (have 'int' and 'struct A')
- return y + func(y ? ((SomeA.X + 40) + SomeA) / 42 + SomeA.X : SomeA.X);
- ^
   $ clang -fsyntax-only t.c
   t.c:7:39: error: invalid operands to binary expression ('int' and 'struct A')
 return y + func(y ? ((SomeA.X + 40) + SomeA) 
/ 42 + SomeA.X : SomeA.X);
@@ -122,7 +116,7 @@ GCC Compatibility
 
 
 GCC is currently the defacto-standard open source compiler today, and it
-routinely compiles a huge volume of code.  GCC supports a huge number of
+routinely compiles a huge volume of code. GCC supports a huge number of
 extensions and features (many of which are undocumented) and a lot of
 code and header files depend on these features in order to build.
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] dac98cf - Fix the links to clang analyzers checkers

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-19T22:31:24+01:00
New Revision: dac98cfa03e9c87a57bd47dfb9880fb01999fbff

URL: 
https://github.com/llvm/llvm-project/commit/dac98cfa03e9c87a57bd47dfb9880fb01999fbff
DIFF: 
https://github.com/llvm/llvm-project/commit/dac98cfa03e9c87a57bd47dfb9880fb01999fbff.diff

LOG: Fix the links to clang analyzers checkers

Added: 


Modified: 
clang-tools-extra/docs/clang-tidy/checks/list.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst 
b/clang-tools-extra/docs/clang-tidy/checks/list.rst
index 1bd1a55bc791..565fe7a03c09 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/list.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst
@@ -109,87 +109,87 @@ Clang-Tidy Checks
cert-oop54-cpp (redirects to bugprone-unhandled-self-assignment) 

cert-oop58-cpp
cert-pos44-c (redirects to bugprone-bad-signal-to-kill-thread) 

-   clang-analyzer-core.CallAndMessage (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.DivideZero (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 
+   clang-analyzer-core.CallAndMessage (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.DivideZero (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

clang-analyzer-core.DynamicTypePropagation
-   clang-analyzer-core.NonNullParamChecker (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.NullDereference (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.StackAddressEscape (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.UndefinedBinaryOperatorResult (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.VLASize (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 
-   clang-analyzer-core.uninitialized.ArraySubscript (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.uninitialized.Assign (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-core.uninitialized.Branch (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

+   clang-analyzer-core.NonNullParamChecker (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.NullDereference (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.StackAddressEscape (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.UndefinedBinaryOperatorResult (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.VLASize (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.uninitialized.ArraySubscript (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.uninitialized.Assign (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-core.uninitialized.Branch (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

clang-analyzer-core.uninitialized.CapturedBlockVariable
-   clang-analyzer-core.uninitialized.UndefReturn (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

+   clang-analyzer-core.uninitialized.UndefReturn (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

clang-analyzer-cplusplus.InnerPointer
-   clang-analyzer-cplusplus.Move (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 
-   clang-analyzer-cplusplus.NewDelete (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-cplusplus.NewDeleteLeaks (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-deadcode.DeadStores (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-nullability.NullPassedToNonnull (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-nullability.NullReturnedFromNonnull (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-nullability.NullableDereferenced (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

-   clang-analyzer-nullability.NullablePassedToNonnull (redirects to 
https://clang.llvm.org/docs/analyzer/checkers) 

+   clang-analyzer-cplusplus.Move (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-cplusplus.NewDelete (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-cplusplus.NewDeleteLeaks (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-deadcode.DeadStores (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

+   clang-analyzer-nullability.NullPassedToNonnull (redirects to 
https://clang.llvm.org/docs/analyzer/checkers.html) 

[clang-tools-extra] 918d393 - Fix some typos in the clang-tools-extra doc

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-19T22:23:35+01:00
New Revision: 918d393972237fe2f9c0f4c7cd14ed4ec4ba706a

URL: 
https://github.com/llvm/llvm-project/commit/918d393972237fe2f9c0f4c7cd14ed4ec4ba706a
DIFF: 
https://github.com/llvm/llvm-project/commit/918d393972237fe2f9c0f4c7cd14ed4ec4ba706a.diff

LOG: Fix some typos in the clang-tools-extra doc

Added: 


Modified: 
clang-tools-extra/docs/clang-doc.rst
clang-tools-extra/docs/clang-include-fixer.rst
clang-tools-extra/docs/clang-tidy/checks/abseil-duration-factory-scale.rst
clang-tools-extra/docs/clang-tidy/checks/abseil-duration-subtraction.rst
clang-tools-extra/docs/clang-tidy/checks/bugprone-branch-clone.rst
clang-tools-extra/docs/clang-tidy/checks/bugprone-exception-escape.rst
clang-tools-extra/docs/clang-tidy/checks/bugprone-fold-init-type.rst
clang-tools-extra/docs/clang-tidy/checks/bugprone-suspicious-enum-usage.rst
clang-tools-extra/docs/clang-tidy/checks/bugprone-unused-return-value.rst
clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
clang-tools-extra/docs/clang-tidy/checks/google-runtime-int.rst
clang-tools-extra/docs/clang-tidy/checks/hicpp-multiway-paths-covered.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-headers.rst
clang-tools-extra/docs/clang-tidy/checks/modernize-use-override.rst

clang-tools-extra/docs/clang-tidy/checks/performance-inefficient-algorithm.rst

clang-tools-extra/docs/clang-tidy/checks/readability-implicit-bool-conversion.rst
clang-tools-extra/docs/clangd/Installation.rst
clang-tools-extra/docs/pp-trace.rst

Removed: 




diff  --git a/clang-tools-extra/docs/clang-doc.rst 
b/clang-tools-extra/docs/clang-doc.rst
index 0c47405fb5a9..93a6f67e2c8f 100644
--- a/clang-tools-extra/docs/clang-doc.rst
+++ b/clang-tools-extra/docs/clang-doc.rst
@@ -7,7 +7,7 @@ Clang-Doc
 .. toctree::
:maxdepth: 1
 
-:program:`clang-doc` is a tool for generating C and C++ documenation from 
+:program:`clang-doc` is a tool for generating C and C++ documentation from 
 source code and comments. 
 
 The tool is in a very early development stage, so you might encounter bugs and
@@ -100,7 +100,7 @@ Options
   generated if the file is in this dir.
 --stylesheets=  - CSS stylesheets to extend the default styles.
 
-The following flags shoud only be used if ``format`` is set to ``html``:
+The following flags should only be used if ``format`` is set to ``html``:
 - ``repository``
 - ``source-root``
 - ``stylesheets``

diff  --git a/clang-tools-extra/docs/clang-include-fixer.rst 
b/clang-tools-extra/docs/clang-include-fixer.rst
index 783f45efe5f2..b934095d8e74 100644
--- a/clang-tools-extra/docs/clang-include-fixer.rst
+++ b/clang-tools-extra/docs/clang-include-fixer.rst
@@ -105,7 +105,7 @@ Customized settings in `.vimrc`:
 
   Set to 1 if you want to insert ``#include`` for the symbol under the cursor.
   Default is 0. Compared to normal mode, this mode won't parse the source file
-  and only search the sysmbol from database, which is faster than normal mode.
+  and only search the symbol from database, which is faster than normal mode.
 
 See ``clang-include-fixer.py`` for more details.
 

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-factory-scale.rst 
b/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-factory-scale.rst
index 8b99d4685f3c..cba520295746 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-factory-scale.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-factory-scale.rst
@@ -5,7 +5,7 @@ abseil-duration-factory-scale
 
 Checks for cases where arguments to ``absl::Duration`` factory functions are
 scaled internally and could be changed to a 
diff erent factory function. This
-check also looks for arguements with a zero value and suggests using
+check also looks for arguments with a zero value and suggests using
 ``absl::ZeroDuration()`` instead.
 
 Examples:

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-subtraction.rst 
b/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-subtraction.rst
index b570931b92b0..3dfd32656208 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-subtraction.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/abseil-duration-subtraction.rst
@@ -30,7 +30,7 @@ Examples:
 
 
 Note: As with other ``clang-tidy`` checks, it is possible that multiple fixes
-may overlap (as in the case of nested expressions), so not all occurences can
+may overlap (as in the case of nested expressions), so not all occurrences can
 be transformed in one run. In particular, this may occur for nested subtraction
 expressions. Running ``clang-tidy`` multiple times will find and fix these
 overlaps.

diff  --git 
a/clang-tools-extra/docs/clang-tidy/checks/bugprone-branch-clone.rst 

[clang] a05d7c2 - Fix typo in the AST Matcher Reference doc Closes: #54

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-12-08T16:14:31+01:00
New Revision: a05d7c278ee2a29aec73dbe5316e5cf2a2d190f8

URL: 
https://github.com/llvm/llvm-project/commit/a05d7c278ee2a29aec73dbe5316e5cf2a2d190f8
DIFF: 
https://github.com/llvm/llvm-project/commit/a05d7c278ee2a29aec73dbe5316e5cf2a2d190f8.diff

LOG: Fix typo in the AST Matcher Reference doc Closes: #54

Added: 


Modified: 
clang/docs/LibASTMatchersReference.html
clang/include/clang/ASTMatchers/ASTMatchers.h

Removed: 




diff  --git a/clang/docs/LibASTMatchersReference.html 
b/clang/docs/LibASTMatchersReference.html
index d9e867c9519d..97c75277c275 100644
--- a/clang/docs/LibASTMatchersReference.html
+++ b/clang/docs/LibASTMatchersReference.html
@@ -5212,7 +5212,7 @@ AST Traversal Matchers
 
 Given:
   MyClass *p1 = new MyClass[10];
-cxxNewExpr(hasArraySize(intgerLiteral(equals(10
+cxxNewExpr(hasArraySize(integerLiteral(equals(10
   matches the expression 'new MyClass[10]'.
 
 

diff  --git a/clang/include/clang/ASTMatchers/ASTMatchers.h 
b/clang/include/clang/ASTMatchers/ASTMatchers.h
index 608454631556..9c0aae2886fc 100644
--- a/clang/include/clang/ASTMatchers/ASTMatchers.h
+++ b/clang/include/clang/ASTMatchers/ASTMatchers.h
@@ -6566,7 +6566,7 @@ AST_MATCHER(CXXNewExpr, isArray) {
 /// \code
 ///   MyClass *p1 = new MyClass[10];
 /// \endcode
-/// cxxNewExpr(hasArraySize(intgerLiteral(equals(10
+/// cxxNewExpr(hasArraySize(integerLiteral(equals(10
 ///   matches the expression 'new MyClass[10]'.
 AST_MATCHER_P(CXXNewExpr, hasArraySize, internal::Matcher, InnerMatcher) 
{
   return Node.isArray() && *Node.getArraySize() &&



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 42effc1 - Add support of the next Ubuntu (Ubuntu 20.04 - Focal Fossa)

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-16T12:21:54+01:00
New Revision: 42effc106966d45053abdd4e3d03ecfe256869b5

URL: 
https://github.com/llvm/llvm-project/commit/42effc106966d45053abdd4e3d03ecfe256869b5
DIFF: 
https://github.com/llvm/llvm-project/commit/42effc106966d45053abdd4e3d03ecfe256869b5.diff

LOG: Add support of the next Ubuntu (Ubuntu 20.04 - Focal Fossa)

Added: 


Modified: 
clang/include/clang/Driver/Distro.h
clang/lib/Driver/Distro.cpp

Removed: 




diff  --git a/clang/include/clang/Driver/Distro.h 
b/clang/include/clang/Driver/Distro.h
index 67dc764fb7d1..da8f819dee96 100644
--- a/clang/include/clang/Driver/Distro.h
+++ b/clang/include/clang/Driver/Distro.h
@@ -65,6 +65,7 @@ class Distro {
 UbuntuCosmic,
 UbuntuDisco,
 UbuntuEoan,
+UbuntuFocal,
 UnknownDistro
   };
 
@@ -118,7 +119,7 @@ class Distro {
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuEoan;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuFocal;
   }
 
   bool IsAlpineLinux() const {

diff  --git a/clang/lib/Driver/Distro.cpp b/clang/lib/Driver/Distro.cpp
index f2a3074d1e70..92e04108a7e2 100644
--- a/clang/lib/Driver/Distro.cpp
+++ b/clang/lib/Driver/Distro.cpp
@@ -52,6 +52,7 @@ static Distro::DistroType DetectDistro(llvm::vfs::FileSystem 
) {
   .Case("cosmic", Distro::UbuntuCosmic)
   .Case("disco", Distro::UbuntuDisco)
   .Case("eoan", Distro::UbuntuEoan)
+  .Case("focal", Distro::UbuntuFocal)
   .Default(Distro::UnknownDistro);
 if (Version != Distro::UnknownDistro)
   return Version;



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 9b40a7f - Remove +x permission on some files

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-16T14:47:20+01:00
New Revision: 9b40a7f3bf7198dd64403cbd83478087e72f994b

URL: 
https://github.com/llvm/llvm-project/commit/9b40a7f3bf7198dd64403cbd83478087e72f994b
DIFF: 
https://github.com/llvm/llvm-project/commit/9b40a7f3bf7198dd64403cbd83478087e72f994b.diff

LOG: Remove +x permission on some files

Added: 


Modified: 
clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp
clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h
clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h
clang/lib/StaticAnalyzer/Checkers/Yaml.h
clang/test/Modules/lsv-debuginfo.cpp

lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
openmp/runtime/test/ompt/callback.h

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp 
b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp
old mode 100755
new mode 100644

diff  --git a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h 
b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h
old mode 100755
new mode 100644

diff  --git 
a/clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h 
b/clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h
old mode 100755
new mode 100644

diff  --git a/clang/lib/StaticAnalyzer/Checkers/Yaml.h 
b/clang/lib/StaticAnalyzer/Checkers/Yaml.h
old mode 100755
new mode 100644

diff  --git a/clang/test/Modules/lsv-debuginfo.cpp 
b/clang/test/Modules/lsv-debuginfo.cpp
old mode 100755
new mode 100644

diff  --git 
a/lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
 
b/lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
old mode 100755
new mode 100644

diff  --git a/openmp/runtime/test/ompt/callback.h 
b/openmp/runtime/test/ompt/callback.h
old mode 100755
new mode 100644



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 9b40a7f - Remove +x permission on some files

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-16T14:47:20+01:00
New Revision: 9b40a7f3bf7198dd64403cbd83478087e72f994b

URL: 
https://github.com/llvm/llvm-project/commit/9b40a7f3bf7198dd64403cbd83478087e72f994b
DIFF: 
https://github.com/llvm/llvm-project/commit/9b40a7f3bf7198dd64403cbd83478087e72f994b.diff

LOG: Remove +x permission on some files

Added: 


Modified: 
clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp
clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h
clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h
clang/lib/StaticAnalyzer/Checkers/Yaml.h
clang/test/Modules/lsv-debuginfo.cpp

lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
openmp/runtime/test/ompt/callback.h

Removed: 




diff  --git a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp 
b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp
old mode 100755
new mode 100644

diff  --git a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h 
b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.h
old mode 100755
new mode 100644

diff  --git 
a/clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h 
b/clang/include/clang/Tooling/Refactoring/Extract/SourceExtraction.h
old mode 100755
new mode 100644

diff  --git a/clang/lib/StaticAnalyzer/Checkers/Yaml.h 
b/clang/lib/StaticAnalyzer/Checkers/Yaml.h
old mode 100755
new mode 100644

diff  --git a/clang/test/Modules/lsv-debuginfo.cpp 
b/clang/test/Modules/lsv-debuginfo.cpp
old mode 100755
new mode 100644

diff  --git 
a/lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
 
b/lldb/packages/Python/lldbsuite/test/functionalities/thread/backtrace_all/ParallelTask.cpp
old mode 100755
new mode 100644

diff  --git a/openmp/runtime/test/ompt/callback.h 
b/openmp/runtime/test/ompt/callback.h
old mode 100755
new mode 100644



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d4819fe - Remove +x permission on clang/lib/Format/Format.cpp

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-16T14:44:35+01:00
New Revision: d4819fe0ec1037e1a995dca911278fee46384922

URL: 
https://github.com/llvm/llvm-project/commit/d4819fe0ec1037e1a995dca911278fee46384922
DIFF: 
https://github.com/llvm/llvm-project/commit/d4819fe0ec1037e1a995dca911278fee46384922.diff

LOG: Remove +x permission on clang/lib/Format/Format.cpp

Added: 


Modified: 
clang/lib/Format/Format.cpp

Removed: 




diff  --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp
old mode 100755
new mode 100644



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 152e83f - clang-format: fix a typo introduced by the previous change

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-11T21:52:08+01:00
New Revision: 152e83fc59af7d255df10c0f56c8fbb14dc1dea2

URL: 
https://github.com/llvm/llvm-project/commit/152e83fc59af7d255df10c0f56c8fbb14dc1dea2
DIFF: 
https://github.com/llvm/llvm-project/commit/152e83fc59af7d255df10c0f56c8fbb14dc1dea2.diff

LOG: clang-format: fix a typo introduced by the previous change

Added: 


Modified: 
clang/docs/ClangFormat.rst

Removed: 




diff  --git a/clang/docs/ClangFormat.rst b/clang/docs/ClangFormat.rst
index 73dbe509efbe..7fe050ec9e29 100644
--- a/clang/docs/ClangFormat.rst
+++ b/clang/docs/ClangFormat.rst
@@ -83,7 +83,7 @@ to format C/C++/Java/JavaScript/Objective-C/Protobuf/C# code.
   Generic Options:
 
 --help - Display available options (--help-hidden for 
more)
---help-list- Display list of available options 
(--help-list-hidden for moOAre)
+--help-list- Display list of available options 
(--help-list-hidden for more)
 --version  - Display the version of this program
 
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 3a3255a - clang-format: refresh the list of options

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-08T23:40:16+01:00
New Revision: 3a3255a22398f34dec16b57beef9e2ff60a108e3

URL: 
https://github.com/llvm/llvm-project/commit/3a3255a22398f34dec16b57beef9e2ff60a108e3
DIFF: 
https://github.com/llvm/llvm-project/commit/3a3255a22398f34dec16b57beef9e2ff60a108e3.diff

LOG: clang-format: refresh the list of options

Added: 


Modified: 
clang/docs/ClangFormat.rst

Removed: 




diff  --git a/clang/docs/ClangFormat.rst b/clang/docs/ClangFormat.rst
index c1347f3070bc..73dbe509efbe 100644
--- a/clang/docs/ClangFormat.rst
+++ b/clang/docs/ClangFormat.rst
@@ -30,54 +30,61 @@ to format C/C++/Java/JavaScript/Objective-C/Protobuf/C# 
code.
 
   Clang-format options:
 
--assume-filename= - When reading from stdin, clang-format assumes 
this
-filename to look for a style config file (with
--style=file) and to determine the language.
--cursor=- The position of the cursor when invoking
-clang-format from an editor integration
--dump-config  - Dump configuration options to stdout and exit.
-Can be used with -style option.
--fallback-style=  - The name of the predefined style used as a
-fallback in case clang-format is invoked with
--style=file, but can not find the .clang-format
-file to use.
-Use -fallback-style=none to skip formatting.
--i- Inplace edit s, if specified.
--length=- Format a range of this length (in bytes).
-Multiple ranges can be formatted by specifying
-several -offset and -length pairs.
-When only a single -offset is specified without
--length, clang-format will format up to the end
-of the file.
-Can only be used with one input file.
--lines=   - : - format a range of
-lines (both 1-based).
-Multiple ranges can be formatted by specifying
-several -lines arguments.
-Can't be used with -offset and -length.
-Can only be used with one input file.
--offset=- Format a range starting at this byte offset.
-Multiple ranges can be formatted by specifying
-several -offset and -length pairs.
-Can only be used with one input file.
--output-replacements-xml  - Output replacements as XML.
--sort-includes- Sort touched include lines
--style=   - Coding style, currently supports:
-  LLVM, Google, Chromium, Mozilla, WebKit.
-Use -style=file to load style configuration 
from
-.clang-format file located in one of the parent
-directories of the source file (or current
-directory for stdin).
-Use -style="{key: value, ...}" to set specific
-parameters, e.g.:
-  -style="{BasedOnStyle: llvm, IndentWidth: 8}"
--verbose  - If set, shows the list of processed files
+--Werror   - If set, changes formatting warnings to errors
+--assume-filename= - When reading from stdin, clang-format assumes 
this
+ filename to look for a style config file (with
+ -style=file) and to determine the language.
+--cursor=- The position of the cursor when invoking
+ clang-format from an editor integration
+--dry-run  - If set, do not actually make the formatting 
changes
+--dump-config  - Dump configuration options to stdout and exit.
+ Can be used with -style option.
+--fallback-style=  - The name of the predefined style used as a
+ fallback in case clang-format is invoked with
+ -style=file, but can not find the 
.clang-format
+ file to use.
+ Use -fallback-style=none to skip formatting.
+--ferror-limit=  - Set the maximum number of clang-format errors 
to
+ emit before stopping (0 = no limit). Used only
+ with --dry-run 

[clang] 4c44fd3 - clang-format: Add to the release notes the new --dry-run/-n option

[Sylvestre Ledru via cfe-commits]

Author: Sylvestre Ledru
Date: 2019-11-08T23:26:44+01:00
New Revision: 4c44fd3de374fdce383ca1b0313b2418d1f1d370

URL: 
https://github.com/llvm/llvm-project/commit/4c44fd3de374fdce383ca1b0313b2418d1f1d370
DIFF: 
https://github.com/llvm/llvm-project/commit/4c44fd3de374fdce383ca1b0313b2418d1f1d370.diff

LOG: clang-format: Add to the release notes the new --dry-run/-n option

Added: 


Modified: 
clang/docs/ReleaseNotes.rst

Removed: 




diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index 51bcd013df6d..3db1603e0631 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -289,6 +289,9 @@ clang-format
   ``Cpp11`` is treated as ``Latest``, as this was always clang-format's 
behavior.
   (One motivation for this change is the new name describes the behavior 
better).
 
+- clang-format gets a new option called ``--dry-run`` or ``-n`` to emit a
+  warning.
+
 libclang
 
 



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r374659 - remove an useless allocation found by scan-build - the new Dead nested assignment check

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Sat Oct 12 08:24:00 2019
New Revision: 374659

URL: http://llvm.org/viewvc/llvm-project?rev=374659=rev
Log:
remove an useless allocation found by scan-build - the new Dead nested 
assignment check

Modified:
cfe/trunk/lib/CodeGen/CGExprScalar.cpp

Modified: cfe/trunk/lib/CodeGen/CGExprScalar.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGExprScalar.cpp?rev=374659=374658=374659=diff
==
--- cfe/trunk/lib/CodeGen/CGExprScalar.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGExprScalar.cpp Sat Oct 12 08:24:00 2019
@@ -4413,8 +4413,8 @@ Value *ScalarExprEmitter::VisitAsTypeExp
 return Src;
   }
 
-  return Src = createCastsForTypeOfSameSize(Builder, CGF.CGM.getDataLayout(),
-Src, DstTy, "astype");
+  return createCastsForTypeOfSameSize(Builder, CGF.CGM.getDataLayout(),
+  Src, DstTy, "astype");
 }
 
 Value *ScalarExprEmitter::VisitAtomicExpr(AtomicExpr *E) {


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r374593 - Release notes: Add the option WarnForDeadNestedAssignments

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Oct 11 13:33:43 2019
New Revision: 374593

URL: http://llvm.org/viewvc/llvm-project?rev=374593=rev
Log:
Release notes: Add the option WarnForDeadNestedAssignments
https://reviews.llvm.org/D66733

Modified:
cfe/trunk/docs/ReleaseNotes.rst

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=374593=374592=374593=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Fri Oct 11 13:33:43 2019
@@ -244,6 +244,9 @@ libclang
 Static Analyzer
 ---
 
+- The Clang analyzer checker ``DeadStores`` gets a new option called
+  ``WarnForDeadNestedAssignments`` to detect nested dead assignments
+  (enabled by default).
 - ...
 
 .. _release-notes-ubsan:


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r374032 - Remove an useless allocation (from by clang-analyzer/scan-build)

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Tue Oct  8 02:17:46 2019
New Revision: 374032

URL: http://llvm.org/viewvc/llvm-project?rev=374032=rev
Log:
Remove an useless allocation (from by clang-analyzer/scan-build)
https://llvm.org/reports/scan-build/report-TargetInfo.cpp-detectFPCCEligibleStruct-9-1.html#EndPath


Modified:
cfe/trunk/lib/CodeGen/TargetInfo.cpp

Modified: cfe/trunk/lib/CodeGen/TargetInfo.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/TargetInfo.cpp?rev=374032=374031=374032=diff
==
--- cfe/trunk/lib/CodeGen/TargetInfo.cpp (original)
+++ cfe/trunk/lib/CodeGen/TargetInfo.cpp Tue Oct  8 02:17:46 2019
@@ -9438,7 +9438,7 @@ bool RISCVABIInfo::detectFPCCEligibleStr
   Ty, CharUnits::Zero(), Field1Ty, Field1Off, Field2Ty, Field2Off);
   // Not really a candidate if we have a single int but no float.
   if (Field1Ty && !Field2Ty && !Field1Ty->isFloatingPointTy())
-return IsCandidate = false;
+return false;
   if (!IsCandidate)
 return false;
   if (Field1Ty && Field1Ty->isFloatingPointTy())


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r373308 - Decrease the verbosity of the -ftime-trace option

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Tue Oct  1 00:52:42 2019
New Revision: 373308

URL: http://llvm.org/viewvc/llvm-project?rev=373308=rev
Log:
Decrease the verbosity of the -ftime-trace option
And move the relevant information in the doc.

Summary:
Currently, building a large software like Firefox shows
'Use chrome://tracing or Speedscope App (https://www.speedscope.app) for 
flamegraph visualization'
for each file.

Reviewers: anton-afanasyev

Reviewed By: anton-afanasyev

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D68260

Modified:
cfe/trunk/docs/ClangCommandLineReference.rst
cfe/trunk/tools/driver/cc1_main.cpp

Modified: cfe/trunk/docs/ClangCommandLineReference.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ClangCommandLineReference.rst?rev=373308=373307=373308=diff
==
--- cfe/trunk/docs/ClangCommandLineReference.rst (original)
+++ cfe/trunk/docs/ClangCommandLineReference.rst Tue Oct  1 00:52:42 2019
@@ -1942,7 +1942,8 @@ Perform ThinLTO importing using provided
 
 .. option:: -ftime-trace
 
-Turn on time profiler
+Turn on time profiler. Results can be analyzed with chrome://tracing or
+`Speedscope App `_ for flamegraph visualization
 
 .. option:: -ftime-trace-granularity=
 

Modified: cfe/trunk/tools/driver/cc1_main.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/driver/cc1_main.cpp?rev=373308=373307=373308=diff
==
--- cfe/trunk/tools/driver/cc1_main.cpp (original)
+++ cfe/trunk/tools/driver/cc1_main.cpp Tue Oct  1 00:52:42 2019
@@ -271,9 +271,6 @@ int cc1_main(ArrayRef Argv
 llvm::timeTraceProfilerCleanup();
 
 llvm::errs() << "Time trace json-file dumped to " << Path.str() << "\n";
-llvm::errs()
-<< "Use chrome://tracing or Speedscope App "
-   "(https://www.speedscope.app) for flamegraph visualization\n";
   }
 
   // Our error handler depends on the Diagnostics object, which we're


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r371832 - Fix a perl warning: Scalar value @ArgParts[0] better written as $ArgParts[0] at /usr/share/clang/scan-build-10/libexec/ccc-analyzer line 502.

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Sep 13 02:31:19 2019
New Revision: 371832

URL: http://llvm.org/viewvc/llvm-project?rev=371832=rev
Log:
Fix a perl warning: Scalar value @ArgParts[0] better written as $ArgParts[0] at 
/usr/share/clang/scan-build-10/libexec/ccc-analyzer line 502.


Modified:
cfe/trunk/tools/scan-build/libexec/ccc-analyzer

Modified: cfe/trunk/tools/scan-build/libexec/ccc-analyzer
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/scan-build/libexec/ccc-analyzer?rev=371832=371831=371832=diff
==
--- cfe/trunk/tools/scan-build/libexec/ccc-analyzer (original)
+++ cfe/trunk/tools/scan-build/libexec/ccc-analyzer Fri Sep 13 02:31:19 2019
@@ -499,7 +499,7 @@ my $HasSDK = 0;
 foreach (my $i = 0; $i < scalar(@ARGV); ++$i) {
   my $Arg = $ARGV[$i];
   my @ArgParts = split /=/,$Arg,2;
-  my $ArgKey = @ArgParts[0];
+  my $ArgKey = $ArgParts[0];
 
   # Be friendly to "" in the argument list.
   if (!defined($ArgKey)) {


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r370671 - ThinLTO: Document the option BOOTSTRAP_LLVM_ENABLE_LTO

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Mon Sep  2 08:34:53 2019
New Revision: 370671

URL: http://llvm.org/viewvc/llvm-project?rev=370671=rev
Log:
ThinLTO: Document the option BOOTSTRAP_LLVM_ENABLE_LTO

Modified:
cfe/trunk/docs/ThinLTO.rst

Modified: cfe/trunk/docs/ThinLTO.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ThinLTO.rst?rev=370671=370670=370671=diff
==
--- cfe/trunk/docs/ThinLTO.rst (original)
+++ cfe/trunk/docs/ThinLTO.rst Mon Sep  2 08:34:53 2019
@@ -225,6 +225,10 @@ To bootstrap clang/LLVM with ThinLTO, fo
``CMAKE_EXE_LINKER_FLAGS:STRING=``. Note the configure may fail if
linker plugin options are instead specified directly in the previous step.
 
+The `BOOTSTRAP_LLVM_ENABLE_LTO=Thin`` will enable ThinLTO for stage 2 and
+stage 3 in case the compiler used for stage 1 does not support the ThinLTO
+option.
+
 More Information
 
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r366091 - Update __VERSION__ to remove the hardcoded 4.2.1 version

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Mon Jul 15 10:47:22 2019
New Revision: 366091

URL: http://llvm.org/viewvc/llvm-project?rev=366091=rev
Log:
Update __VERSION__ to remove the hardcoded 4.2.1 version

Summary:
Just like in https://reviews.llvm.org/D56803
for -dumpversion

Reviewers: rnk

Reviewed By: rnk

Subscribers: dexonsmith, lebedev.ri, hubert.reinterpretcast, xbolva00, 
fedor.sergeev, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D63048

Modified:
cfe/trunk/docs/LanguageExtensions.rst
cfe/trunk/docs/ReleaseNotes.rst
cfe/trunk/lib/Frontend/InitPreprocessor.cpp
cfe/trunk/test/Preprocessor/init.c

Modified: cfe/trunk/docs/LanguageExtensions.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/LanguageExtensions.rst?rev=366091=366090=366091=diff
==
--- cfe/trunk/docs/LanguageExtensions.rst (original)
+++ cfe/trunk/docs/LanguageExtensions.rst Mon Jul 15 10:47:22 2019
@@ -324,6 +324,8 @@ option for a warning and returns true if
   ...
   #endif
 
+.. _languageextensions-builtin-macros:
+
 Builtin Macros
 ==
 

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=366091=366090=366091=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Mon Jul 15 10:47:22 2019
@@ -56,6 +56,12 @@ Improvements to Clang's diagnostics
 Non-comprehensive list of changes in this release
 -
 
+- The ``__VERSION__`` macro has been updated.
+  Previously this macro contained the string '4.2.1 Compatible' to achieve
+  compatibility with GCC 4.2.1, but that should no longer be necessary.
+  However, to retrieve Clang's version, please favor the one of the macro
+  defined in :ref:`clang namespaced version macros 
`.
+
 - ...
 
 

Modified: cfe/trunk/lib/Frontend/InitPreprocessor.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitPreprocessor.cpp?rev=366091=366090=366091=diff
==
--- cfe/trunk/lib/Frontend/InitPreprocessor.cpp (original)
+++ cfe/trunk/lib/Frontend/InitPreprocessor.cpp Mon Jul 15 10:47:22 2019
@@ -604,10 +604,9 @@ static void InitializePredefinedMacros(c
   // Support for #pragma redefine_extname (Sun compatibility)
   Builder.defineMacro("__PRAGMA_REDEFINE_EXTNAME", "1");
 
-  // As sad as it is, enough software depends on the __VERSION__ for version
-  // checks that it is necessary to report 4.2.1 (the base GCC version we claim
-  // compatibility with) first.
-  Builder.defineMacro("__VERSION__", "\"4.2.1 Compatible " +
+  // Previously this macro was set to a string aiming to achieve compatibility
+  // with GCC 4.2.1. Now, just return the full Clang version
+  Builder.defineMacro("__VERSION__", "\"" +
   Twine(getClangFullCPPVersion()) + "\"");
 
   // Initialize language-specific preprocessor defines.

Modified: cfe/trunk/test/Preprocessor/init.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Preprocessor/init.c?rev=366091=366090=366091=diff
==
--- cfe/trunk/test/Preprocessor/init.c (original)
+++ cfe/trunk/test/Preprocessor/init.c Mon Jul 15 10:47:22 2019
@@ -8169,7 +8169,7 @@
 // SPARC:#define __UINT_LEAST8_MAX__ 255
 // SPARC:#define __UINT_LEAST8_TYPE__ unsigned char
 // SPARC:#define __USER_LABEL_PREFIX__
-// SPARC:#define __VERSION__ "4.2.1 Compatible{{.*}}
+// SPARC:#define __VERSION__ "Clang{{.*}}
 // SPARC:#define __WCHAR_MAX__ 2147483647
 // SPARC:#define __WCHAR_TYPE__ int
 // SPARC:#define __WCHAR_WIDTH__ 32
@@ -9041,7 +9041,7 @@
 // X86_64-CLOUDABI:#define __UINT_LEAST8_MAX__ 255
 // X86_64-CLOUDABI:#define __UINT_LEAST8_TYPE__ unsigned char
 // X86_64-CLOUDABI:#define __USER_LABEL_PREFIX__
-// X86_64-CLOUDABI:#define __VERSION__ "4.2.1 Compatible{{.*}}
+// X86_64-CLOUDABI:#define __VERSION__ "Clang{{.*}}
 // X86_64-CLOUDABI:#define __WCHAR_MAX__ 2147483647
 // X86_64-CLOUDABI:#define __WCHAR_TYPE__ int
 // X86_64-CLOUDABI:#define __WCHAR_WIDTH__ 32


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r366024 - consistency in the release notes

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Sun Jul 14 11:25:09 2019
New Revision: 366024

URL: http://llvm.org/viewvc/llvm-project?rev=366024=rev
Log:
consistency in the release notes

Modified:
cfe/trunk/docs/ReleaseNotes.rst

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=366024=366023=366024=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Sun Jul 14 11:25:09 2019
@@ -184,8 +184,8 @@ AST Matchers
 clang-format
 
 
-- Add language support for clang-formatting C# files
-- Add Microsoft coding style to encapsulate default C# formatting style
+- Add language support for clang-formatting C# files.
+- Add Microsoft coding style to encapsulate default C# formatting style.
 - Added new option `PPDIS_BeforeHash` (in configuration: `BeforeHash`) to
   `IndentPPDirectives` which indents preprocessor directives before the hash.
 - Added new option `AlignConsecutiveMacros` to align the C/C++ preprocessor


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r365992 - This reverts commit 632a36bfcfc8273c1861f04ff6758d863c47c784.

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Jul 12 23:27:35 2019
New Revision: 365992

URL: http://llvm.org/viewvc/llvm-project?rev=365992=rev
Log:
This reverts commit 632a36bfcfc8273c1861f04ff6758d863c47c784.

Some targets such as Python 2.7.16 still use VERSION in
their builds. Without VERSION defined, the source code
has syntax errors.

Reverting as it will probably break many other things.

Noticed by Sterling Augustine


Modified:
cfe/trunk/docs/LanguageExtensions.rst
cfe/trunk/docs/ReleaseNotes.rst
cfe/trunk/lib/Basic/Version.cpp
cfe/trunk/lib/Frontend/InitPreprocessor.cpp
cfe/trunk/test/Index/complete-exprs.c
cfe/trunk/test/Preprocessor/init.c
cfe/trunk/utils/builtin-defines.c

Modified: cfe/trunk/docs/LanguageExtensions.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/LanguageExtensions.rst?rev=365992=365991=365992=diff
==
--- cfe/trunk/docs/LanguageExtensions.rst (original)
+++ cfe/trunk/docs/LanguageExtensions.rst Fri Jul 12 23:27:35 2019
@@ -324,8 +324,6 @@ option for a warning and returns true if
   ...
   #endif
 
-.. _languageextensions-builtin-macros:
-
 Builtin Macros
 ==
 

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=365992=365991=365992=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Fri Jul 12 23:27:35 2019
@@ -56,11 +56,6 @@ Improvements to Clang's diagnostics
 Non-comprehensive list of changes in this release
 -
 
-- The ``__VERSION__`` macro has been removed.
-  Previously this macro was set to a string aiming to achieve compatibility 
with 
-  GCC 4.2.1, but that should no longer be necessary. To get Clang's version,
-  use the :ref:`clang namespaced version macros 
`.
-
 - ...
 
 
@@ -84,11 +79,6 @@ Modified Compiler Flags
 
 - ...
 
-Removed Compiler Options
-
-
-- ...
-
 New Pragmas in Clang
 
 

Modified: cfe/trunk/lib/Basic/Version.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Version.cpp?rev=365992=365991=365992=diff
==
--- cfe/trunk/lib/Basic/Version.cpp (original)
+++ cfe/trunk/lib/Basic/Version.cpp Fri Jul 12 23:27:35 2019
@@ -136,6 +136,8 @@ std::string getClangToolFullVersion(Stri
 }
 
 std::string getClangFullCPPVersion() {
+  // The version string we report in __VERSION__ is just a compacted version of
+  // the one we report on the command line.
   std::string buf;
   llvm::raw_string_ostream OS(buf);
 #ifdef CLANG_VENDOR

Modified: cfe/trunk/lib/Frontend/InitPreprocessor.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitPreprocessor.cpp?rev=365992=365991=365992=diff
==
--- cfe/trunk/lib/Frontend/InitPreprocessor.cpp (original)
+++ cfe/trunk/lib/Frontend/InitPreprocessor.cpp Fri Jul 12 23:27:35 2019
@@ -604,6 +604,12 @@ static void InitializePredefinedMacros(c
   // Support for #pragma redefine_extname (Sun compatibility)
   Builder.defineMacro("__PRAGMA_REDEFINE_EXTNAME", "1");
 
+  // As sad as it is, enough software depends on the __VERSION__ for version
+  // checks that it is necessary to report 4.2.1 (the base GCC version we claim
+  // compatibility with) first.
+  Builder.defineMacro("__VERSION__", "\"4.2.1 Compatible " +
+  Twine(getClangFullCPPVersion()) + "\"");
+
   // Initialize language-specific preprocessor defines.
 
   // Standard conforming mode?

Modified: cfe/trunk/test/Index/complete-exprs.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/complete-exprs.c?rev=365992=365991=365992=diff
==
--- cfe/trunk/test/Index/complete-exprs.c (original)
+++ cfe/trunk/test/Index/complete-exprs.c Fri Jul 12 23:27:35 2019
@@ -27,6 +27,7 @@ void f5(float f) {
 // RUN: c-index-test -code-completion-at=%s:7:10 -Xclang 
-code-completion-patterns %s | FileCheck -check-prefix=CHECK-CC1 %s
 // RUN: env CINDEXTEST_EDITING=1 CINDEXTEST_COMPLETION_CACHING=1 c-index-test 
-code-completion-at=%s:7:10 -Xclang -code-completion-patterns %s | FileCheck 
-check-prefix=CHECK-CC1 %s
 // CHECK-CC1: NotImplemented:{TypedText __PRETTY_FUNCTION__} (65)
+// CHECK-CC1: macro definition:{TypedText __VERSION__} (70)
 // CHECK-CC1: FunctionDecl:{ResultType int}{TypedText f}{LeftParen 
(}{Placeholder int}{RightParen )} (12) (unavailable)
 // CHECK-CC1-NOT: NotImplemented:{TypedText float} (65)
 // CHECK-CC1: ParmDecl:{ResultType int}{TypedText j} (8)
@@ -38,6 +39,7 @@ void f5(float f) {
 // RUN: c-index-test -code-completion-at=%s:7:18 -Xclang 
-code-completion-patterns %s | FileCheck 

r365962 - Remove __VERSION__

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Jul 12 14:45:08 2019
New Revision: 365962

URL: http://llvm.org/viewvc/llvm-project?rev=365962=rev
Log:
Remove __VERSION__

Summary:
It has been introduced in 2011 for gcc compat:
https://github.com/llvm-mirror/clang/commit/ad1a4c6e89594e704775ddb6b036ac982fd68cad
it is probably time to remove it


Reviewers: rnk, dexonsmith

Reviewed By: rnk

Subscribers: dschuff, aheejin, fedor.sergeev, arphaman, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64062

Modified:
cfe/trunk/docs/LanguageExtensions.rst
cfe/trunk/docs/ReleaseNotes.rst
cfe/trunk/lib/Basic/Version.cpp
cfe/trunk/lib/Frontend/InitPreprocessor.cpp
cfe/trunk/test/Index/complete-exprs.c
cfe/trunk/test/Preprocessor/init.c
cfe/trunk/utils/builtin-defines.c

Modified: cfe/trunk/docs/LanguageExtensions.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/LanguageExtensions.rst?rev=365962=365961=365962=diff
==
--- cfe/trunk/docs/LanguageExtensions.rst (original)
+++ cfe/trunk/docs/LanguageExtensions.rst Fri Jul 12 14:45:08 2019
@@ -324,6 +324,8 @@ option for a warning and returns true if
   ...
   #endif
 
+.. _languageextensions-builtin-macros:
+
 Builtin Macros
 ==
 

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=365962=365961=365962=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Fri Jul 12 14:45:08 2019
@@ -56,6 +56,11 @@ Improvements to Clang's diagnostics
 Non-comprehensive list of changes in this release
 -
 
+- The ``__VERSION__`` macro has been removed.
+  Previously this macro was set to a string aiming to achieve compatibility 
with 
+  GCC 4.2.1, but that should no longer be necessary. To get Clang's version,
+  use the :ref:`clang namespaced version macros 
`.
+
 - ...
 
 
@@ -79,6 +84,11 @@ Modified Compiler Flags
 
 - ...
 
+Removed Compiler Options
+
+
+- ...
+
 New Pragmas in Clang
 
 

Modified: cfe/trunk/lib/Basic/Version.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Version.cpp?rev=365962=365961=365962=diff
==
--- cfe/trunk/lib/Basic/Version.cpp (original)
+++ cfe/trunk/lib/Basic/Version.cpp Fri Jul 12 14:45:08 2019
@@ -136,8 +136,6 @@ std::string getClangToolFullVersion(Stri
 }
 
 std::string getClangFullCPPVersion() {
-  // The version string we report in __VERSION__ is just a compacted version of
-  // the one we report on the command line.
   std::string buf;
   llvm::raw_string_ostream OS(buf);
 #ifdef CLANG_VENDOR

Modified: cfe/trunk/lib/Frontend/InitPreprocessor.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitPreprocessor.cpp?rev=365962=365961=365962=diff
==
--- cfe/trunk/lib/Frontend/InitPreprocessor.cpp (original)
+++ cfe/trunk/lib/Frontend/InitPreprocessor.cpp Fri Jul 12 14:45:08 2019
@@ -604,12 +604,6 @@ static void InitializePredefinedMacros(c
   // Support for #pragma redefine_extname (Sun compatibility)
   Builder.defineMacro("__PRAGMA_REDEFINE_EXTNAME", "1");
 
-  // As sad as it is, enough software depends on the __VERSION__ for version
-  // checks that it is necessary to report 4.2.1 (the base GCC version we claim
-  // compatibility with) first.
-  Builder.defineMacro("__VERSION__", "\"4.2.1 Compatible " +
-  Twine(getClangFullCPPVersion()) + "\"");
-
   // Initialize language-specific preprocessor defines.
 
   // Standard conforming mode?

Modified: cfe/trunk/test/Index/complete-exprs.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/complete-exprs.c?rev=365962=365961=365962=diff
==
--- cfe/trunk/test/Index/complete-exprs.c (original)
+++ cfe/trunk/test/Index/complete-exprs.c Fri Jul 12 14:45:08 2019
@@ -27,7 +27,6 @@ void f5(float f) {
 // RUN: c-index-test -code-completion-at=%s:7:10 -Xclang 
-code-completion-patterns %s | FileCheck -check-prefix=CHECK-CC1 %s
 // RUN: env CINDEXTEST_EDITING=1 CINDEXTEST_COMPLETION_CACHING=1 c-index-test 
-code-completion-at=%s:7:10 -Xclang -code-completion-patterns %s | FileCheck 
-check-prefix=CHECK-CC1 %s
 // CHECK-CC1: NotImplemented:{TypedText __PRETTY_FUNCTION__} (65)
-// CHECK-CC1: macro definition:{TypedText __VERSION__} (70)
 // CHECK-CC1: FunctionDecl:{ResultType int}{TypedText f}{LeftParen 
(}{Placeholder int}{RightParen )} (12) (unavailable)
 // CHECK-CC1-NOT: NotImplemented:{TypedText float} (65)
 // CHECK-CC1: ParmDecl:{ResultType int}{TypedText j} (8)
@@ -39,7 +38,6 @@ void f5(float f) {
 // RUN: 

r365831 - Rename libclang_shared to libclang-cpp

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Thu Jul 11 14:42:55 2019
New Revision: 365831

URL: http://llvm.org/viewvc/llvm-project?rev=365831=rev
Log:
Rename libclang_shared to libclang-cpp

Summary: Fix bug 42475

Reviewers: beanz, tstellar

Reviewed By: beanz

Subscribers: kimgr, mgorny, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64278

Modified:
cfe/trunk/CMakeLists.txt
cfe/trunk/cmake/modules/AddClang.cmake
cfe/trunk/tools/clang-shlib/CMakeLists.txt

Modified: cfe/trunk/CMakeLists.txt
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/CMakeLists.txt?rev=365831=365830=365831=diff
==
--- cfe/trunk/CMakeLists.txt (original)
+++ cfe/trunk/CMakeLists.txt Thu Jul 11 14:42:55 2019
@@ -324,7 +324,7 @@ set(CLANG_PYTHON_BINDINGS_VERSIONS "" CA
 "Python versions to install libclang python bindings for")
 
 set(CLANG_LINK_CLANG_DYLIB ${LLVM_LINK_LLVM_DYLIB} CACHE BOOL
-"Link tools against libclang_shared.so")
+"Link tools against libclang-cpp.so")
 
 if (NOT LLVM_LINK_LLVM_DYLIB AND CLANG_LINK_CLANG_DYLIB)
   message(FATAL_ERROR "Cannot set CLANG_LINK_CLANG_DYLIB=ON when "

Modified: cfe/trunk/cmake/modules/AddClang.cmake
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/cmake/modules/AddClang.cmake?rev=365831=365830=365831=diff
==
--- cfe/trunk/cmake/modules/AddClang.cmake (original)
+++ cfe/trunk/cmake/modules/AddClang.cmake Thu Jul 11 14:42:55 2019
@@ -175,7 +175,7 @@ endmacro()
 
 function(clang_target_link_libraries target type)
   if (CLANG_LINK_CLANG_DYLIB)
-target_link_libraries(${target} ${type} clang_shared)
+target_link_libraries(${target} ${type} clang-cpp)
   else()
 target_link_libraries(${target} ${type} ${ARGN})
   endif()

Modified: cfe/trunk/tools/clang-shlib/CMakeLists.txt
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/clang-shlib/CMakeLists.txt?rev=365831=365830=365831=diff
==
--- cfe/trunk/tools/clang-shlib/CMakeLists.txt (original)
+++ cfe/trunk/tools/clang-shlib/CMakeLists.txt Thu Jul 11 14:42:55 2019
@@ -1,4 +1,4 @@
-# Building libclang_shared.so fails if LLVM_ENABLE_PIC=Off
+# Building libclang-cpp.so fails if LLVM_ENABLE_PIC=Off
 if (NOT LLVM_ENABLE_PIC)
   return()
 endif()
@@ -38,7 +38,7 @@ foreach (lib ${clang_libs})
   list(APPEND _DEPS ${lib})
 endforeach ()
 
-add_clang_library(clang_shared
+add_clang_library(clang-cpp
   SHARED
   clang-shlib.cpp
   ${_OBJECTS}


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r365446 - Remove trailing whitespaces in the Language Extensions doc

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Tue Jul  9 01:50:17 2019
New Revision: 365446

URL: http://llvm.org/viewvc/llvm-project?rev=365446=rev
Log:
Remove trailing whitespaces in the Language Extensions doc

Modified:
cfe/trunk/docs/LanguageExtensions.rst

Modified: cfe/trunk/docs/LanguageExtensions.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/LanguageExtensions.rst?rev=365446=365445=365446=diff
==
--- cfe/trunk/docs/LanguageExtensions.rst (original)
+++ cfe/trunk/docs/LanguageExtensions.rst Tue Jul  9 01:50:17 2019
@@ -330,11 +330,11 @@ Builtin Macros
 ``__BASE_FILE__``
   Defined to a string that contains the name of the main input file passed to
   Clang.
-  
+
 ``__FILE_NAME__``
   Clang-specific extension that functions similar to ``__FILE__`` but only
   renders the last path component (the filename) instead of an invocation
-  dependent full path to that file. 
+  dependent full path to that file.
 
 ``__COUNTER__``
   Defined to an integer value that starts at zero and is incremented each time
@@ -1372,7 +1372,7 @@ Objective-C retaining behavior attribute
 -
 
 In Objective-C, functions and methods are generally assumed to follow the
-`Cocoa Memory Management 
+`Cocoa Memory Management
 
`_
 conventions for ownership of object arguments and
 return values. However, there are exceptions, and so Clang provides attributes
@@ -1789,7 +1789,7 @@ the bitpattern of an integer value; for
 **Description**:
 
 The '``__builtin_rotateleft``' family of builtins is used to rotate
-the bits in the first argument by the amount in the second argument. 
+the bits in the first argument by the amount in the second argument.
 For example, ``0b1110`` rotated left by 11 becomes ``0b00110100``.
 The shift value is treated as an unsigned amount modulo the size of
 the arguments. Both arguments and the result have the bitwidth specified
@@ -1821,7 +1821,7 @@ by the name of the builtin.
 **Description**:
 
 The '``__builtin_rotateright``' family of builtins is used to rotate
-the bits in the first argument by the amount in the second argument. 
+the bits in the first argument by the amount in the second argument.
 For example, ``0b1110`` rotated right by 3 becomes ``0b1101``.
 The shift value is treated as an unsigned amount modulo the size of
 the arguments. Both arguments and the result have the bitwidth specified
@@ -2143,8 +2143,8 @@ Atomic Min/Max builtins with memory orde
 There are two atomic builtins with min/max in-memory comparison and swap.
 The syntax and semantics are similar to GCC-compatible __atomic_* builtins.
 
-* ``__atomic_fetch_min`` 
-* ``__atomic_fetch_max`` 
+* ``__atomic_fetch_min``
+* ``__atomic_fetch_max``
 
 The builtins work with signed and unsigned integers and require to specify 
memory ordering.
 The return value is the original value that was stored in memory before 
comparison.
@@ -2262,7 +2262,7 @@ C++ Coroutines support builtins
 
 
 .. warning::
-  This is a work in progress. Compatibility across Clang/LLVM releases is not 
+  This is a work in progress. Compatibility across Clang/LLVM releases is not
   guaranteed.
 
 Clang provides experimental builtins to support C++ Coroutines as defined by
@@ -2310,7 +2310,7 @@ Other coroutine builtins are either for
 development of the coroutine feature. See `Coroutines in LLVM
 `_ for
 more information on their semantics. Note that builtins matching the intrinsics
-that take token as the first parameter (llvm.coro.begin, llvm.coro.alloc, 
+that take token as the first parameter (llvm.coro.begin, llvm.coro.alloc,
 llvm.coro.free and llvm.coro.suspend) omit the token parameter and fill it to
 an appropriate value during the emission.
 
@@ -2485,8 +2485,8 @@ The effective values for mode are:
 - 2 - round to +infinity
 - 3 - round to -infinity
 
-Note that the mode argument will modulo 4, so if the int argument is greater 
-than 3, it will only use the least significant two bits of the mode. 
+Note that the mode argument will modulo 4, so if the int argument is greater
+than 3, it will only use the least significant two bits of the mode.
 Namely, ``__builtin_setrnd(102))`` is equal to ``__builtin_setrnd(2)``.
 
 PowerPC Language Extensions
@@ -2510,7 +2510,7 @@ The effective values for mode are:
 - 3 - round to -infinity
 
 Note that the mode argument will modulo 4, so if the integer argument is 
greater
-than 3, it will only use the least significant two bits of the mode. 
+than 3, it will only use the least significant two bits of the mode.
 Namely, ``__builtin_setrnd(102))`` is equal to ``__builtin_setrnd(2)``.
 
 PowerPC Language Extensions
@@ -2534,7 +2534,7 @@ The effective values for mode are:
 - 3 - round to 

r365445 - Add AlignConsecutiveMacros to the clang release notes

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Tue Jul  9 01:45:55 2019
New Revision: 365445

URL: http://llvm.org/viewvc/llvm-project?rev=365445=rev
Log:
Add AlignConsecutiveMacros to the clang release notes

Modified:
cfe/trunk/docs/ReleaseNotes.rst

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=365445=365444=365445=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Tue Jul  9 01:45:55 2019
@@ -188,6 +188,8 @@ clang-format
 - Add Microsoft coding style to encapsulate default C# formatting style
 - Added new option `PPDIS_BeforeHash` (in configuration: `BeforeHash`) to
   `IndentPPDirectives` which indents preprocessor directives before the hash.
+- Added new option `AlignConsecutiveMacros` to align the C/C++ preprocessor
+  macros of consecutive lines.
 
 libclang
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D61909: Add Clang shared library with C++ exports

[Sylvestre Ledru via cfe-commits]

libclangcpp ?

I think it is a pretty common pattern.

On debian, apt-cache search 'lib.*cpp'  returns a bunch of libraries 
(libhdf5-cpp, libroscpp2d, libjsonrpccpp-dev, libmysqlcppconn7v5,

libsvncpp3, libtercpp0v5, libyaml-cpp-dev, etc)

S


Le 02/07/2019 à 01:22, Chris Bieneman a écrit :

The question is, what *should* it be called.

While yes, the 's' in 'so' is shared, the "dylib" and "dll" extensions 
on Darwin and Windows have the same meaning too. The problem is 
libclang.so is already taken.


I'm not attached to the name in any way, so I'm open to suggestions.

We do have documentation on best practices for how to build 
distributions, which includes explanations of how to pick and choose 
what you want to install 
(http://llvm.org/docs/BuildingADistribution.html), so you shouldn't 
need an option to disable it.


-Chris

On Jul 1, 2019, at 6:13 PM, Tom Stellard via Phabricator 
mailto:revi...@reviews.llvm.org>> wrote:


tstellar added a comment.

In D61909#1563678 , 
@sylvestre.ledru wrote:



For now, it isn't part of the debian packaging.
https://salsa.debian.org/pkg-llvm-team/llvm-toolchain/blob/snapshot/debian/rules#L563
it is removed as packaging phase as I have been told it isn't ready.

Anyway, the lib should not keep this name.
By definition, on linux, .so means shared. It should have a more 
explicit name.


@beanz Can you please rename it? thanks



I've filed a bug for this and marked it as a blocker for 9.0.0, 
because once we ship a release with this name, it will be harder to 
change: https://bugs.llvm.org/show_bug.cgi?id=42475



Repository:
 rC Clang

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

https://reviews.llvm.org/D61909





___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r363369 - add a missing parenthese in the clang doc

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Jun 14 04:33:40 2019
New Revision: 363369

URL: http://llvm.org/viewvc/llvm-project?rev=363369=rev
Log:
add a missing parenthese in the clang doc

Modified:
cfe/trunk/docs/ThinLTO.rst

Modified: cfe/trunk/docs/ThinLTO.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ThinLTO.rst?rev=363369=363368=363369=diff
==
--- cfe/trunk/docs/ThinLTO.rst (original)
+++ cfe/trunk/docs/ThinLTO.rst Fri Jun 14 04:33:40 2019
@@ -198,7 +198,7 @@ To bootstrap clang/LLVM with ThinLTO, fo
 
 1. The host compiler_ must be a version of clang that supports ThinLTO.
 #. The host linker_ must support ThinLTO (and in the case of gold, must be
-   `configured with plugins enabled `_.
+   `configured with plugins enabled `_).
 #. Use the following additional `CMake variables
`_
when configuring the bootstrap compiler build:


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r359350 - Add to the release notes the fact that UninitializedObject checker is now

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Apr 26 14:46:33 2019
New Revision: 359350

URL: http://llvm.org/viewvc/llvm-project?rev=359350=rev
Log:
Add to the release notes the fact that UninitializedObject checker is now
considered as stable


Modified:
cfe/trunk/docs/ReleaseNotes.rst

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=359350=359349=359350=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Fri Apr 26 14:46:33 2019
@@ -190,7 +190,8 @@ libclang
 Static Analyzer
 ---
 
-- ...
+- The UninitializedObject checker is now considered as stable.
+  (moved from the 'alpha.cplusplus' to the 'optin.cplusplus' package)
 
 ...
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r358758 - Debian: Add two missing version code in sid

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Apr 19 06:48:52 2019
New Revision: 358758

URL: http://llvm.org/viewvc/llvm-project?rev=358758=rev
Log:
Debian: Add two missing version code in sid

Modified:
cfe/trunk/lib/Driver/Distro.cpp

Modified: cfe/trunk/lib/Driver/Distro.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Distro.cpp?rev=358758=358757=358758=diff
==
--- cfe/trunk/lib/Driver/Distro.cpp (original)
+++ cfe/trunk/lib/Driver/Distro.cpp Fri Apr 19 06:48:52 2019
@@ -105,6 +105,8 @@ static Distro::DistroType DetectDistro(l
 .Case("wheezy/sid", Distro::DebianWheezy)
 .Case("jessie/sid", Distro::DebianJessie)
 .Case("stretch/sid", Distro::DebianStretch)
+.Case("buster/sid", Distro::DebianBuster)
+.Case("bullseye/sid", Distro::DebianBullseye)
 .Default(Distro::UnknownDistro);
   }
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r358757 - Add support of the future Debian (Debian 11 - Bullseye)

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Apr 19 06:46:58 2019
New Revision: 358757

URL: http://llvm.org/viewvc/llvm-project?rev=358757=rev
Log:
Add support of the future Debian (Debian 11 - Bullseye)
https://wiki.debian.org/DebianBullseye


Modified:
cfe/trunk/include/clang/Driver/Distro.h
cfe/trunk/lib/Driver/Distro.cpp

Modified: cfe/trunk/include/clang/Driver/Distro.h
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Distro.h?rev=358757=358756=358757=diff
==
--- cfe/trunk/include/clang/Driver/Distro.h (original)
+++ cfe/trunk/include/clang/Driver/Distro.h Fri Apr 19 06:46:58 2019
@@ -33,6 +33,7 @@ public:
 DebianJessie,
 DebianStretch,
 DebianBuster,
+DebianBullseye,
 Exherbo,
 RHEL5,
 RHEL6,
@@ -113,7 +114,7 @@ public:
   }
 
   bool IsDebian() const {
-return DistroVal >= DebianLenny && DistroVal <= DebianBuster;
+return DistroVal >= DebianLenny && DistroVal <= DebianBullseye;
   }
 
   bool IsUbuntu() const {

Modified: cfe/trunk/lib/Driver/Distro.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Distro.cpp?rev=358757=358756=358757=diff
==
--- cfe/trunk/lib/Driver/Distro.cpp (original)
+++ cfe/trunk/lib/Driver/Distro.cpp Fri Apr 19 06:46:58 2019
@@ -94,6 +94,8 @@ static Distro::DistroType DetectDistro(l
 return Distro::DebianStretch;
   case 10:
 return Distro::DebianBuster;
+  case 11:
+return Distro::DebianBullseye;
   default:
 return Distro::UnknownDistro;
   }


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r358756 - Add support of the next Ubuntu (Ubuntu 19.10 - Eoan EANIMAL)

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Fri Apr 19 06:43:28 2019
New Revision: 358756

URL: http://llvm.org/viewvc/llvm-project?rev=358756=rev
Log:
Add support of the next Ubuntu (Ubuntu 19.10 - Eoan EANIMAL)


Modified:
cfe/trunk/include/clang/Driver/Distro.h
cfe/trunk/lib/Driver/Distro.cpp

Modified: cfe/trunk/include/clang/Driver/Distro.h
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Distro.h?rev=358756=358755=358756=diff
==
--- cfe/trunk/include/clang/Driver/Distro.h (original)
+++ cfe/trunk/include/clang/Driver/Distro.h Fri Apr 19 06:43:28 2019
@@ -63,6 +63,7 @@ public:
 UbuntuBionic,
 UbuntuCosmic,
 UbuntuDisco,
+UbuntuEoan,
 UnknownDistro
   };
 
@@ -116,7 +117,7 @@ public:
   }
 
   bool IsUbuntu() const {
-return DistroVal >= UbuntuHardy && DistroVal <= UbuntuDisco;
+return DistroVal >= UbuntuHardy && DistroVal <= UbuntuEoan;
   }
 
   bool IsAlpineLinux() const {

Modified: cfe/trunk/lib/Driver/Distro.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Distro.cpp?rev=358756=358755=358756=diff
==
--- cfe/trunk/lib/Driver/Distro.cpp (original)
+++ cfe/trunk/lib/Driver/Distro.cpp Fri Apr 19 06:43:28 2019
@@ -51,6 +51,7 @@ static Distro::DistroType DetectDistro(l
   .Case("bionic", Distro::UbuntuBionic)
   .Case("cosmic", Distro::UbuntuCosmic)
   .Case("disco", Distro::UbuntuDisco)
+  .Case("eoan", Distro::UbuntuEoan)
   .Default(Distro::UnknownDistro);
 if (Version != Distro::UnknownDistro)
   return Version;


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r357566 - Adding 'CLion integration' to clang-format doc

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Wed Apr  3 01:56:30 2019
New Revision: 357566

URL: http://llvm.org/viewvc/llvm-project?rev=357566=rev
Log:
Adding 'CLion integration' to clang-format doc

Summary:
This commit adds a chapter 'CLion integration' to ClangFormat.rst. 
The official announcement of clang-format support in CLion 2019.1: 
https://blog.jetbrains.com/clion/2019/03/clion-2019-1-embedded-dev-clangformat-memory-view/

Patch by Marina Kalashina 

Reviewers: djasper, sylvestre.ledru, krasimir

Reviewed By: sylvestre.ledru

Subscribers: jdoerfert, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D60115

Modified:
cfe/trunk/docs/ClangFormat.rst

Modified: cfe/trunk/docs/ClangFormat.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ClangFormat.rst?rev=357566=357565=357566=diff
==
--- cfe/trunk/docs/ClangFormat.rst (original)
+++ cfe/trunk/docs/ClangFormat.rst Wed Apr  3 01:56:30 2019
@@ -165,6 +165,19 @@ menu item by renaming the script, and ca
 shortcut in the BBEdit preferences, under Menus & Shortcuts.
 
 
+CLion Integration
+==
+
+:program:`clang-format` is integrated into `CLion `_ as an alternative code formatter. It is disabled by default and
+can be turned on in Settings/Preferences | Editor | Code Style.
+
+If :program:`clang-format` support is enabled, CLion detects config files when
+opening a project and suggests overriding the current IDE settings. Code style
+rules from the ``.clang-format`` files are then applied automatically to all
+editor actions, including auto-completion, code generation, and refactorings.
+
+
 Visual Studio Integration
 =
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] r357371 - Spelling correction for docs for cppcoreguidelines-owning-memory

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Sun Mar 31 14:53:00 2019
New Revision: 357371

URL: http://llvm.org/viewvc/llvm-project?rev=357371=rev
Log:
Spelling correction for docs for cppcoreguidelines-owning-memory

Summary: There's a typo in the docs, as mentioned in the title. Please see the 
diff.

Reviewers: JonasToth

Subscribers: sylvestre.ledru, nemanjai, kbarton, cfe-commits

Tags: #clang-tools-extra, #clang

Differential Revision: https://reviews.llvm.org/D60050

Modified:

clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst

Modified: 
clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
URL: 
http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst?rev=357371=357370=357371=diff
==
--- 
clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
 (original)
+++ 
clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
 Sun Mar 31 14:53:00 2019
@@ -50,7 +50,7 @@ to be deleted.
   int* NonOwner = new int(42); // First warning here, since new must land in 
an owner
   delete NonOwner; // Second warning here, since only owners are allowed to be 
deleted
 
-  // Example Good, Ownership correclty stated
+  // Example Good, Ownership correctly stated
   gsl::owner Owner = new int(42); // Good
   delete Owner; // Good as well, statically enforced, that only owners get 
deleted
   


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r356983 - update the release notes after the change of 'clang -dumpversion'

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Tue Mar 26 02:48:23 2019
New Revision: 356983

URL: http://llvm.org/viewvc/llvm-project?rev=356983=rev
Log:
update the release notes after the change of 'clang -dumpversion'

Modified:
cfe/trunk/docs/ReleaseNotes.rst

Modified: cfe/trunk/docs/ReleaseNotes.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/ReleaseNotes.rst?rev=356983=356982=356983=diff
==
--- cfe/trunk/docs/ReleaseNotes.rst (original)
+++ cfe/trunk/docs/ReleaseNotes.rst Tue Mar 26 02:48:23 2019
@@ -75,6 +75,8 @@ future versions of Clang.
 Modified Compiler Flags
 ---
 
+- `clang -dumpversion` now returns the version of Clang itself.
+
 - ...
 
 New Pragmas in Clang


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r356931 - clang -dumpversion returns 4.2.1 for legacy reason, update it

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Mon Mar 25 11:06:20 2019
New Revision: 356931

URL: http://llvm.org/viewvc/llvm-project?rev=356931=rev
Log:
clang -dumpversion returns 4.2.1 for legacy reason, update it

Summary:
It has been introduced in 2011 for gcc compat:
https://github.com/llvm-mirror/clang/commit/95a907fc0f2f717dca38a379c1d2353bfea06d4f
it is probably time to remove it to remove the confusion.

reported: https://bugs.llvm.org/show_bug.cgi?id=38836
https://bugs.launchpad.net/ubuntu/+source/llvm-defaults/+bug/1810860

Reviewers: ddunbar, rnk

Reviewed By: rnk

Subscribers: jdoerfert, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D56803

Modified:
cfe/trunk/lib/Driver/Driver.cpp
cfe/trunk/test/Driver/immediate-options.c

Modified: cfe/trunk/lib/Driver/Driver.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Driver.cpp?rev=356931=356930=356931=diff
==
--- cfe/trunk/lib/Driver/Driver.cpp (original)
+++ cfe/trunk/lib/Driver/Driver.cpp Mon Mar 25 11:06:20 2019
@@ -1640,11 +1640,7 @@ bool Driver::HandleImmediateArgs(const C
   if (C.getArgs().hasArg(options::OPT_dumpversion)) {
 // Since -dumpversion is only implemented for pedantic GCC compatibility, 
we
 // return an answer which matches our definition of __VERSION__.
-//
-// If we want to return a more correct answer some day, then we should
-// introduce a non-pedantically GCC compatible mode to Clang in which we
-// provide sensible definitions for -dumpversion, __VERSION__, etc.
-llvm::outs() << "4.2.1\n";
+llvm::outs() << CLANG_VERSION_STRING << "\n";
 return false;
   }
 

Modified: cfe/trunk/test/Driver/immediate-options.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/immediate-options.c?rev=356931=356930=356931=diff
==
--- cfe/trunk/test/Driver/immediate-options.c (original)
+++ cfe/trunk/test/Driver/immediate-options.c Mon Mar 25 11:06:20 2019
@@ -7,7 +7,7 @@
 // HELP-HIDDEN: driver-mode
 
 // RUN: %clang -dumpversion | FileCheck %s -check-prefix=DUMPVERSION
-// DUMPVERSION: 4.2.1
+// DUMPVERSION: {{[0-9]+\.[0-9.]+}}
 
 // RUN: %clang -print-search-dirs | FileCheck %s 
-check-prefix=PRINT-SEARCH-DIRS
 // PRINT-SEARCH-DIRS: programs: ={{.*}}


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r356842 - Sync some doc changes ClangFormatStyleOptions.rst with doc comments in `Format.h`

[Sylvestre Ledru via cfe-commits]

Author: sylvestre
Date: Sat Mar 23 10:57:31 2019
New Revision: 356842

URL: http://llvm.org/viewvc/llvm-project?rev=356842=rev
Log:
Sync some doc changes ClangFormatStyleOptions.rst with doc comments in 
`Format.h`

Summary:
These changes were corrected directly in ClangFormatStyleOptions.rst (llvm-svn: 
350192 and llvm-svn: 351976) but these sections can be produced automatically 
using `dump_format_style.py` so sync the corresponding doc comments in 
`Format.h` as well.

Patch by Ronald Wampler

Reviewers: eugene, sylvestre.ledru, djasper

Reviewed By: sylvestre.ledru

Subscribers: jdoerfert, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D58186

Modified:
cfe/trunk/include/clang/Format/Format.h
cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h

Modified: cfe/trunk/include/clang/Format/Format.h
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Format/Format.h?rev=356842=356841=356842=diff
==
--- cfe/trunk/include/clang/Format/Format.h (original)
+++ cfe/trunk/include/clang/Format/Format.h Sat Mar 23 10:57:31 2019
@@ -1201,7 +1201,7 @@ struct FormatStyle {
 
   /// A vector of prefixes ordered by the desired groups for Java imports.
   ///
-  /// Each group is seperated by a newline. Static imports will also follow the
+  /// Each group is separated by a newline. Static imports will also follow the
   /// same grouping convention above all non-static imports. One group's prefix
   /// can be a subset of another - the longest prefix is always matched. Within
   /// a group, the imports are ordered lexicographically.

Modified: cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h?rev=356842=356841=356842=diff
==
--- cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h (original)
+++ cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h Sat Mar 23 
10:57:31 2019
@@ -67,7 +67,7 @@ struct IncludeStyle {
   /// used for ordering ``#includes``.
   ///
   /// `POSIX extended
-  /// 
`_
+  /// 
`_
   /// regular expressions are supported.
   ///
   /// These regular expressions are matched against the filename of an include
@@ -79,7 +79,7 @@ struct IncludeStyle {
   /// If none of the regular expressions match, INT_MAX is assigned as
   /// category. The main header for a source file automatically gets category 
0.
   /// so that it is generally kept at the beginning of the ``#includes``
-  /// (http://llvm.org/docs/CodingStandards.html#include-style). However, you
+  /// (https://llvm.org/docs/CodingStandards.html#include-style). However, you
   /// can also assign negative priorities if you have certain headers that
   /// always need to be first.
   ///


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits