[libc-commits] [libc] Update LLVMLibCArchitectures.cmake (PR #112464)

via libc-commits libc-commits at lists.llvm.org
Tue Oct 15 20:13:36 PDT 2024


https://github.com/westtide created https://github.com/llvm/llvm-project/pull/112464

Hi there,

When building llvm-libc on the openEuler system, I encountered an issue as shown in the image below:
![image](https://github.com/user-attachments/assets/75667de4-5bea-4a95-be28-ed34db0e05b9)

This issue happens because the regular expression used in `libc/cmake/modules/LLVMLibCArchitectures.cmake`: `string(REGEX MATCH "Target: [-_a-z0-9.]+[ \r\n]+")` does not handle capital letters properly in `openEuler`. 

To fix this, I modified the regular expression to: `string(REGEX MATCH "Target: [-_a-zA-Z0-9.]+[ \r\n]+")`. This change makes it compatible with capital letters.


>From ddfb66d3cf3cfe07dffcefa288f1217519e4ab98 Mon Sep 17 00:00:00 2001
From: westtide <tocokeo at outlook.com>
Date: Wed, 16 Oct 2024 11:07:27 +0800
Subject: [PATCH] Update LLVMLibCArchitectures.cmake

Fix the matching failure of triples containing uppercase latter, such as 'openEuler'
---
 libc/cmake/modules/LLVMLibCArchitectures.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libc/cmake/modules/LLVMLibCArchitectures.cmake b/libc/cmake/modules/LLVMLibCArchitectures.cmake
index 7711127c1a81e1..1e5ed723194a26 100644
--- a/libc/cmake/modules/LLVMLibCArchitectures.cmake
+++ b/libc/cmake/modules/LLVMLibCArchitectures.cmake
@@ -84,7 +84,7 @@ if(NOT (libc_compiler_info_result EQUAL "0"))
   message(FATAL_ERROR "libc build: error querying compiler info from the "
                       "compiler: ${libc_compiler_info}")
 endif()
-string(REGEX MATCH "Target: [-_a-z0-9.]+[ \r\n]+"
+string(REGEX MATCH "Target: [-_a-zA-Z0-9.]+[ \r\n]+"
        libc_compiler_target_info ${libc_compiler_info})
 if(NOT libc_compiler_target_info)
   message(FATAL_ERROR "libc build: could not read compiler target info from:\n"



More information about the libc-commits mailing list