[clang] [NFC][Clang][Headers] Update refs to ACLE in comments (PR #66662)

[Volodymyr Turanskyy via cfe-commits]

@@ -32,7 +32,7 @@ extern "C" {
 #define __isb(i) __builtin_arm_isb(i)
 #endif
 
-/* 8.4 Hints */
+/* 7.4 Hints */

voltur01 wrote:

HTML version has the extra `.` 
https://arm-software.github.io/acle/main/acle.html#hints but the PDF one does 
not 
https://github.com/ARM-software/acle/releases/download/r2023Q2/acle-2023Q2.pdf 
- the solution, probably, depends on what is linked as suggested in a comment 
above: HTML or PDF version.

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

[clang] Multilib support for libraries with exceptions (PR #75031)

[Volodymyr Turanskyy via cfe-commits]

@@ -264,6 +273,15 @@ ToolChain::getMultilibFlags(const llvm::opt::ArgList 
&Args) const {
 break;
   }
 
+  // Include fno-exceptions and fno-rtti
+  // to improve multilib selection
+  if (getRTTIMode() == ToolChain::RTTIMode::RM_Disabled) {
+Result.push_back("-fno-rtti");

voltur01 wrote:

RTTI is required for exceptions support, e.g. a relevant discussion 
https://stackoverflow.com/questions/10320072/is-there-a-relation-between-rtti-and-exceptions

Equally, we can update the title of the PR to reflect both options.

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

[clang] [NFC][Clang][Headers] Update refs to ACLE in comments (PR #78305)

[Volodymyr Turanskyy via cfe-commits]

https://github.com/voltur01 commented:

Looks good to me, thank you for picking it up!

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