[PATCH] D111617: [RISCV] Lazily add RVV C intrinsics.

[Hsiangkai Wang via Phabricator via cfe-commits]

HsiangKai added a comment.

In D111617#3175167 <https://reviews.llvm.org/D111617#3175167>, @craig.topper wrote:

> In D111617#3060377 <https://reviews.llvm.org/D111617#3060377>, @HsiangKai wrote:
>
>> Although it reduces the header size, this patch will increase the binary size of clang.
>>
>> Debug build:
>> Before this patch:
>>
>>   text            data            bss             dec             hex             filename
>>   263892591       10838284        500232          275231107       1067b183        clang-14
>>
>> After this patch:
>>
>>   text            data            bss             dec             hex             filename
>>   263909721       12085116        500232          276495069       107afadd        clang-14
>>
>> Release build:
>> Before this patch;
>>
>>   text            data            bss             dec             hex             filename
>>   382952171       88029764        10264736        481246671       1caf3dcf        clang-14
>>
>> After this patch:
>>
>>   text            data            bss             dec             hex             filename
>>   387629483       94652582        10264736        492546801       1d5baaf1        clang-14
>
> These number indicate the release build is larger than debug. That seems wrong. Am I misreading these numbers?

Indeed, it is strange. The data is got from the first draft version. I have no such environment now. I could collect the clang size using the latest implementation.
After removing the global static constructor, there is no build time problem now.
So, in the latest implementation, we can get the better compile time, smaller clang size and acceptable build time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111617