Skip to content

Attempt to install ClangFormat binaries using CMake #1739

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 20, 2025

Conversation

jviotti
Copy link
Member

@jviotti jviotti commented Jun 20, 2025

Signed-off-by: Juan Cruz Viotti [email protected]

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark (linux/llvm)

Benchmark suite Current: 660f3bb Previous: 2a658ae Ratio
Regex_Lower_S_Or_Upper_S_Asterisk 2.182006580948942 ns/iter 2.1848217510703916 ns/iter 1.00
Regex_Caret_Lower_S_Or_Upper_S_Asterisk_Dollar 2.1810204276120007 ns/iter 2.1863563416565213 ns/iter 1.00
Regex_Period_Asterisk 2.1782510515703244 ns/iter 2.193731233551879 ns/iter 0.99
Regex_Group_Period_Asterisk_Group 2.1819706961514993 ns/iter 2.184043692709391 ns/iter 1.00
Regex_Period_Plus 2.487689279739471 ns/iter 2.4877358046399203 ns/iter 1.00
Regex_Period 2.492643617852567 ns/iter 2.487152176896531 ns/iter 1.00
Regex_Caret_Period_Plus_Dollar 2.4888565743351467 ns/iter 2.486727577909284 ns/iter 1.00
Regex_Caret_Group_Period_Plus_Group_Dollar 2.487746144995843 ns/iter 2.490724892539365 ns/iter 1.00
Regex_Caret_Period_Asterisk_Dollar 3.418436294437521 ns/iter 3.417574809829852 ns/iter 1.00
Regex_Caret_Group_Period_Asterisk_Group_Dollar 3.417142889699351 ns/iter 3.42173171915368 ns/iter 1.00
Regex_Caret_X_Hyphen 12.652916219871384 ns/iter 12.741595984078206 ns/iter 0.99
Regex_Period_Md_Dollar 73.70885895062565 ns/iter 73.47726536379525 ns/iter 1.00
Regex_Caret_Slash_Period_Asterisk 7.1657556286647335 ns/iter 6.21641781443471 ns/iter 1.15
Regex_Caret_Period_Range_Dollar 4.041720881614758 ns/iter 4.045510956363268 ns/iter 1.00
Regex_Nested_Backtrack 466.8151816018052 ns/iter 466.9552402986509 ns/iter 1.00
JSON_Array_Of_Objects_Unique 491.5419489986013 ns/iter 491.5152951859927 ns/iter 1.00
JSON_Parse_1 30256.83099569927 ns/iter 30715.91656542953 ns/iter 0.99
JSON_Fast_Hash_Helm_Chart_Lock 68.32850879180266 ns/iter 68.79722435877272 ns/iter 0.99
JSON_Equality_Helm_Chart_Lock 141.32468067517053 ns/iter 142.67923230324894 ns/iter 0.99
JSON_String_Equal/10 5.914504577879702 ns/iter 6.2238059786111695 ns/iter 0.95
JSON_String_Equal/100 6.530529009418566 ns/iter 6.847154689189083 ns/iter 0.95
JSON_String_Equal_Small_By_Perfect_Hash/10 0.9345465509176174 ns/iter 0.9347242771574776 ns/iter 1.00
JSON_String_Equal_Small_By_Runtime_Perfect_Hash/10 10.257697907027563 ns/iter 10.26101341532297 ns/iter 1.00
JSON_String_Fast_Hash/10 2.7979190773800724 ns/iter 2.79877149794228 ns/iter 1.00
JSON_String_Fast_Hash/100 2.79621692196124 ns/iter 2.7971171439014926 ns/iter 1.00
JSON_String_Key_Hash/10 1.8669886305903367 ns/iter 1.8677574761352491 ns/iter 1.00
JSON_String_Key_Hash/100 2.8119533193740254 ns/iter 2.8010393274281267 ns/iter 1.00
JSON_Object_Defines_Miss_Same_Length 3.751153630620745 ns/iter 3.732806374712164 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Small 3.741464342868018 ns/iter 3.7446995482555367 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Large 3.732986391334158 ns/iter 3.748525178506419 ns/iter 1.00
Pointer_Object_Traverse 44.51049848204818 ns/iter 44.74477333180618 ns/iter 0.99
Pointer_Object_Try_Traverse 52.264456612089916 ns/iter 52.35911717650952 ns/iter 1.00
Pointer_Push_Back_Pointer_To_Weak_Pointer 345.0457381840904 ns/iter 345.54690488833955 ns/iter 1.00
Schema_Frame_OMC_Instances 174493608.74999797 ns/iter 177121638.25000006 ns/iter 0.99
Schema_Frame_OMC_References 97267496.71428284 ns/iter 97389726.2857122 ns/iter 1.00
Schema_Frame_OMC_Locations 94136706.14286016 ns/iter 96228221.00000608 ns/iter 0.98
Schema_Bundle_Meta_2020_12 5865261.403359069 ns/iter 5856982.59322327 ns/iter 1.00

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark (macos/llvm)

Benchmark suite Current: 660f3bb Previous: 2a658ae Ratio
Regex_Lower_S_Or_Upper_S_Asterisk 1.6833160693906153 ns/iter 1.567911881852724 ns/iter 1.07
Regex_Caret_Lower_S_Or_Upper_S_Asterisk_Dollar 1.6807502999126145 ns/iter 1.580011050383762 ns/iter 1.06
Regex_Period_Asterisk 1.6567032052112654 ns/iter 1.571083971200721 ns/iter 1.05
Regex_Group_Period_Asterisk_Group 1.625367234532004 ns/iter 1.5989084721822198 ns/iter 1.02
Regex_Period_Plus 1.9926372899145057 ns/iter 1.9033580642522314 ns/iter 1.05
Regex_Period 1.9262817238229482 ns/iter 1.94878040551403 ns/iter 0.99
Regex_Caret_Period_Plus_Dollar 1.9225936218965576 ns/iter 1.9383629955477917 ns/iter 0.99
Regex_Caret_Group_Period_Plus_Group_Dollar 1.9463378485528893 ns/iter 1.9012578984119095 ns/iter 1.02
Regex_Caret_Period_Asterisk_Dollar 1.5943264864894335 ns/iter 1.570388373292366 ns/iter 1.02
Regex_Caret_Group_Period_Asterisk_Group_Dollar 1.6278777908839615 ns/iter 1.5818668561016853 ns/iter 1.03
Regex_Caret_X_Hyphen 6.359672740901581 ns/iter 6.266982601187419 ns/iter 1.01
Regex_Period_Md_Dollar 68.69028310109142 ns/iter 67.53453604286366 ns/iter 1.02
Regex_Caret_Slash_Period_Asterisk 5.519847950786231 ns/iter 4.8403962040662885 ns/iter 1.14
Regex_Caret_Period_Range_Dollar 2.147725913306858 ns/iter 2.0631455994454693 ns/iter 1.04
Regex_Nested_Backtrack 757.8244479474637 ns/iter 728.1046322462948 ns/iter 1.04
JSON_Array_Of_Objects_Unique 369.4910836245401 ns/iter 341.84473928201504 ns/iter 1.08
JSON_Parse_1 21988.682591857316 ns/iter 21589.93433648373 ns/iter 1.02
JSON_Fast_Hash_Helm_Chart_Lock 51.184641118301926 ns/iter 47.81217555884149 ns/iter 1.07
JSON_Equality_Helm_Chart_Lock 134.8545164938502 ns/iter 126.66491689854986 ns/iter 1.06
JSON_String_Equal/10 8.107935676228001 ns/iter 7.5206071619982735 ns/iter 1.08
JSON_String_Equal/100 6.11599142325803 ns/iter 5.995228391516088 ns/iter 1.02
JSON_String_Equal_Small_By_Perfect_Hash/10 0.3190432431121332 ns/iter 0.31328323156268284 ns/iter 1.02
JSON_String_Equal_Small_By_Runtime_Perfect_Hash/10 3.027203619588049 ns/iter 2.935534383905035 ns/iter 1.03
JSON_String_Fast_Hash/10 1.6438415241616262 ns/iter 1.5945368989542434 ns/iter 1.03
JSON_String_Fast_Hash/100 1.9639338118469556 ns/iter 1.9001327433691395 ns/iter 1.03
JSON_String_Key_Hash/10 1.3339447275590506 ns/iter 1.2547324345048454 ns/iter 1.06
JSON_String_Key_Hash/100 1.7621270084427352 ns/iter 1.6771211236986896 ns/iter 1.05
JSON_Object_Defines_Miss_Same_Length 2.3492006095355253 ns/iter 2.200031714694791 ns/iter 1.07
JSON_Object_Defines_Miss_Too_Small 2.293964676134602 ns/iter 2.216263033635951 ns/iter 1.04
JSON_Object_Defines_Miss_Too_Large 2.311307778864572 ns/iter 2.1975799350203262 ns/iter 1.05
Pointer_Object_Traverse 16.99983001224257 ns/iter 16.248747051661997 ns/iter 1.05
Pointer_Object_Try_Traverse 22.629006627619585 ns/iter 21.712124772535965 ns/iter 1.04
Pointer_Push_Back_Pointer_To_Weak_Pointer 176.16996584293298 ns/iter 169.75362662090572 ns/iter 1.04
Schema_Frame_OMC_Instances 142004591.79999143 ns/iter 137604099.99999863 ns/iter 1.03
Schema_Frame_OMC_References 88000286.49999093 ns/iter 72868641.69999489 ns/iter 1.21
Schema_Frame_OMC_Locations 82655759.2222217 ns/iter 69921291.70000681 ns/iter 1.18
Schema_Bundle_Meta_2020_12 4593080.043799549 ns/iter 4035690.5144569487 ns/iter 1.14

This comment was automatically generated by workflow using github-action-benchmark.

@bavulapati
Copy link
Contributor

@jviotti From my experience with ClangFormat and clang-tidy, it's easier to install llvm.
llvm brings both clang-format and clang-tidy.
On my new Mac, I had to install llvm to get clang-tidy.
brew can easily install llvm.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark (windows/msvc)

Benchmark suite Current: 660f3bb Previous: 2a658ae Ratio
Regex_Lower_S_Or_Upper_S_Asterisk 2.7326999082737626 ns/iter 2.4957278772358493 ns/iter 1.09
Regex_Caret_Lower_S_Or_Upper_S_Asterisk_Dollar 3.102470535714425 ns/iter 3.105433035713645 ns/iter 1.00
Regex_Period_Asterisk 2.797338614821567 ns/iter 2.790006427325858 ns/iter 1.00
Regex_Group_Period_Asterisk_Group 2.7660857130503738 ns/iter 2.4838296417484123 ns/iter 1.11
Regex_Period_Plus 2.7980288826778463 ns/iter 2.787643525541363 ns/iter 1.00
Regex_Period 2.798590579107764 ns/iter 2.789348302325712 ns/iter 1.00
Regex_Caret_Period_Plus_Dollar 2.7998203099997983 ns/iter 2.786408903761941 ns/iter 1.00
Regex_Caret_Group_Period_Plus_Group_Dollar 2.8036354439247355 ns/iter 2.8329002665922 ns/iter 0.99
Regex_Caret_Period_Asterisk_Dollar 2.7930523648253494 ns/iter 2.789946159468437 ns/iter 1.00
Regex_Caret_Group_Period_Asterisk_Group_Dollar 2.6982749083051334 ns/iter 2.487711071428781 ns/iter 1.08
Regex_Caret_X_Hyphen 5.8912044642868615 ns/iter 5.91145535714231 ns/iter 1.00
Regex_Period_Md_Dollar 151.20674107141747 ns/iter 153.93493303572015 ns/iter 0.98
Regex_Caret_Slash_Period_Asterisk 5.8889758928566085 ns/iter 6.023236999999426 ns/iter 0.98
Regex_Caret_Period_Range_Dollar 3.1126374999997455 ns/iter 3.1089482142862477 ns/iter 1.00
Regex_Nested_Backtrack 590.2320535714774 ns/iter 591.4865178571713 ns/iter 1.00
JSON_Array_Of_Objects_Unique 483.6511641265257 ns/iter 482.7810188622315 ns/iter 1.00
JSON_Parse_1 78368.70535718228 ns/iter 86958.37685816595 ns/iter 0.90
JSON_Fast_Hash_Helm_Chart_Lock 62.718464285710844 ns/iter 67.44065000000319 ns/iter 0.93
JSON_Equality_Helm_Chart_Lock 186.64405237893837 ns/iter 186.55090311722867 ns/iter 1.00
JSON_String_Equal/10 8.850387014061702 ns/iter 8.674580357142604 ns/iter 1.02
JSON_String_Equal/100 10.236628124999925 ns/iter 10.22237004364921 ns/iter 1.00
JSON_String_Equal_Small_By_Perfect_Hash/10 2.1685124999990535 ns/iter 2.165814841332608 ns/iter 1.00
JSON_String_Equal_Small_By_Runtime_Perfect_Hash/10 14.101821428563914 ns/iter 14.68621205357274 ns/iter 0.96
JSON_String_Fast_Hash/10 3.719489659366743 ns/iter 3.7152224933656557 ns/iter 1.00
JSON_String_Fast_Hash/100 3.7199951719276574 ns/iter 3.7389969576081503 ns/iter 0.99
JSON_String_Key_Hash/10 7.9620615716002705 ns/iter 8.252132106018388 ns/iter 0.96
JSON_String_Key_Hash/100 4.343636875000811 ns/iter 4.335473125000533 ns/iter 1.00
JSON_Object_Defines_Miss_Same_Length 4.657491974683057 ns/iter 4.647655859124015 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Small 3.7215107076416833 ns/iter 3.714313386224342 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Large 4.804521439296348 ns/iter 4.668189519349049 ns/iter 1.03
Pointer_Object_Traverse 49.46731 ns/iter 50.15889999999672 ns/iter 0.99
Pointer_Object_Try_Traverse 68.90094866070585 ns/iter 68.68838392857616 ns/iter 1.00
Pointer_Push_Back_Pointer_To_Weak_Pointer 161.95535714282576 ns/iter 173.08204060818707 ns/iter 0.94
Schema_Frame_OMC_Instances 371559950.000119 ns/iter 385828699.99993366 ns/iter 0.96
Schema_Frame_OMC_References 222751533.33339405 ns/iter 228286233.33331656 ns/iter 0.98
Schema_Frame_OMC_Locations 214379300.00001872 ns/iter 219210233.3333423 ns/iter 0.98
Schema_Bundle_Meta_2020_12 13641019.999968193 ns/iter 13999486.000016076 ns/iter 0.97

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark (macos/gcc)

Benchmark suite Current: 660f3bb Previous: 2a658ae Ratio
Regex_Lower_S_Or_Upper_S_Asterisk 2.0406983294050933 ns/iter 1.8839327796071934 ns/iter 1.08
Regex_Caret_Lower_S_Or_Upper_S_Asterisk_Dollar 2.1885263977216134 ns/iter 1.8809775146526044 ns/iter 1.16
Regex_Period_Asterisk 2.239778495593682 ns/iter 1.8842337661976551 ns/iter 1.19
Regex_Group_Period_Asterisk_Group 2.2209388906571204 ns/iter 1.881723756508926 ns/iter 1.18
Regex_Period_Plus 1.794900069578853 ns/iter 1.5964344716281986 ns/iter 1.12
Regex_Period 1.7200782929614462 ns/iter 1.591235331675024 ns/iter 1.08
Regex_Caret_Period_Plus_Dollar 1.7162736683284465 ns/iter 1.5772067455977554 ns/iter 1.09
Regex_Caret_Group_Period_Plus_Group_Dollar 1.7502718843610703 ns/iter 1.5999041792642246 ns/iter 1.09
Regex_Caret_Period_Asterisk_Dollar 2.0630114046283405 ns/iter 1.8810248678515917 ns/iter 1.10
Regex_Caret_Group_Period_Asterisk_Group_Dollar 2.0491484891791845 ns/iter 1.8891160706007735 ns/iter 1.08
Regex_Caret_X_Hyphen 6.453943989950701 ns/iter 6.11476387543149 ns/iter 1.06
Regex_Period_Md_Dollar 72.40761433063439 ns/iter 68.9480237123942 ns/iter 1.05
Regex_Caret_Slash_Period_Asterisk 4.74784633717688 ns/iter 4.391114300028308 ns/iter 1.08
Regex_Caret_Period_Range_Dollar 2.0554456929988945 ns/iter 1.8830928467814165 ns/iter 1.09
Regex_Nested_Backtrack 852.9609405617499 ns/iter 787.4434588297194 ns/iter 1.08
JSON_Array_Of_Objects_Unique 218.58903512600173 ns/iter 206.92162158673978 ns/iter 1.06
JSON_Parse_1 27630.227777359985 ns/iter 22563.825940094328 ns/iter 1.22
JSON_Fast_Hash_Helm_Chart_Lock 26.816933841915457 ns/iter 24.0603737047724 ns/iter 1.11
JSON_Equality_Helm_Chart_Lock 123.51403450472407 ns/iter 113.4080814650271 ns/iter 1.09
JSON_String_Equal/10 5.992569737059004 ns/iter 5.417293217951807 ns/iter 1.11
JSON_String_Equal/100 5.5483498842402295 ns/iter 5.056270964179095 ns/iter 1.10
JSON_String_Equal_Small_By_Perfect_Hash/10 0.8273429801163102 ns/iter 0.7451202174184938 ns/iter 1.11
JSON_String_Equal_Small_By_Runtime_Perfect_Hash/10 4.038188127136177 ns/iter 3.791884556045695 ns/iter 1.06
JSON_String_Fast_Hash/10 2.0332173782626897 ns/iter 1.8907541921846454 ns/iter 1.08
JSON_String_Fast_Hash/100 2.055504844851569 ns/iter 1.8915255322915396 ns/iter 1.09
JSON_String_Key_Hash/10 1.7363637495955735 ns/iter 1.6082925657467946 ns/iter 1.08
JSON_String_Key_Hash/100 2.706202193811586 ns/iter 2.521507070572674 ns/iter 1.07
JSON_Object_Defines_Miss_Same_Length 2.0753420296156815 ns/iter 1.7259355654500206 ns/iter 1.20
JSON_Object_Defines_Miss_Too_Small 2.0317366091456726 ns/iter 1.8831101514991593 ns/iter 1.08
JSON_Object_Defines_Miss_Too_Large 1.8773219546687603 ns/iter 1.7483225259943185 ns/iter 1.07
Pointer_Object_Traverse 55.00772274824597 ns/iter 53.257882549759834 ns/iter 1.03
Pointer_Object_Try_Traverse 38.70715945605119 ns/iter 35.7126958762259 ns/iter 1.08
Pointer_Push_Back_Pointer_To_Weak_Pointer 166.44934917695485 ns/iter 157.07076746323867 ns/iter 1.06
Schema_Frame_OMC_Instances 163995742.79785156 ns/iter 143380463.12332153 ns/iter 1.14
Schema_Frame_OMC_References 94114712.30643137 ns/iter 81926643.84841919 ns/iter 1.15
Schema_Frame_OMC_Locations 89573621.74987793 ns/iter 83223316.40455458 ns/iter 1.08
Schema_Bundle_Meta_2020_12 4707448.2358735185 ns/iter 4196145.577345066 ns/iter 1.12

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark (linux/gcc)

Benchmark suite Current: 660f3bb Previous: 2a658ae Ratio
Schema_Frame_OMC_Instances 175559908.33329815 ns/iter 173362479.00001922 ns/iter 1.01
Schema_Frame_OMC_References 98116895.57143056 ns/iter 95701190.1428554 ns/iter 1.03
Schema_Frame_OMC_Locations 94787928.99998163 ns/iter 91118711.99999654 ns/iter 1.04
Schema_Bundle_Meta_2020_12 5913685.666657454 ns/iter 5792506.280997733 ns/iter 1.02
Pointer_Object_Traverse 50.492088684343244 ns/iter 50.14544141573782 ns/iter 1.01
Pointer_Object_Try_Traverse 23.459091485024512 ns/iter 23.378124847545475 ns/iter 1.00
Pointer_Push_Back_Pointer_To_Weak_Pointer 176.39246225317945 ns/iter 194.34291587936391 ns/iter 0.91
JSON_Array_Of_Objects_Unique 406.93094122080754 ns/iter 403.41048032188604 ns/iter 1.01
JSON_Parse_1 32920.31856411272 ns/iter 33335.3151024315 ns/iter 0.99
JSON_Fast_Hash_Helm_Chart_Lock 68.69795711657446 ns/iter 68.76745448558509 ns/iter 1.00
JSON_Equality_Helm_Chart_Lock 150.18387181809175 ns/iter 146.0434586738518 ns/iter 1.03
JSON_String_Equal/10 6.30296503055765 ns/iter 6.299963099916981 ns/iter 1.00
JSON_String_Equal/100 6.946393084335669 ns/iter 6.9551872818077864 ns/iter 1.00
JSON_String_Equal_Small_By_Perfect_Hash/10 0.9341321790827627 ns/iter 0.9343510174711237 ns/iter 1.00
JSON_String_Equal_Small_By_Runtime_Perfect_Hash/10 19.290227292608687 ns/iter 19.302624045461208 ns/iter 1.00
JSON_String_Fast_Hash/10 0.9343653756845228 ns/iter 0.9343454312335524 ns/iter 1.00
JSON_String_Fast_Hash/100 0.9332548114969422 ns/iter 0.9456409473232709 ns/iter 0.99
JSON_String_Key_Hash/10 1.2457160714182003 ns/iter 1.2449573299280998 ns/iter 1.00
JSON_String_Key_Hash/100 2.1796029011501465 ns/iter 2.178285940429226 ns/iter 1.00
JSON_Object_Defines_Miss_Same_Length 2.4980418714142942 ns/iter 2.4898369899839348 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Small 2.8004394744641083 ns/iter 2.797578389223587 ns/iter 1.00
JSON_Object_Defines_Miss_Too_Large 2.8012274894137237 ns/iter 2.801873507889638 ns/iter 1.00
Regex_Lower_S_Or_Upper_S_Asterisk 3.4269493953661057 ns/iter 3.4198866186709873 ns/iter 1.00
Regex_Caret_Lower_S_Or_Upper_S_Asterisk_Dollar 3.433145696213664 ns/iter 3.419064108893135 ns/iter 1.00
Regex_Period_Asterisk 3.443333579850189 ns/iter 3.4273122733521104 ns/iter 1.00
Regex_Group_Period_Asterisk_Group 3.4332821536307647 ns/iter 3.457564579936234 ns/iter 0.99
Regex_Period_Plus 3.740135385245079 ns/iter 3.4203571980413576 ns/iter 1.09
Regex_Period 3.7374440604895023 ns/iter 3.420523716949477 ns/iter 1.09
Regex_Caret_Period_Plus_Dollar 3.7322293567380953 ns/iter 3.423942848576685 ns/iter 1.09
Regex_Caret_Group_Period_Plus_Group_Dollar 3.6301447981343213 ns/iter 3.4237455925195266 ns/iter 1.06
Regex_Caret_Period_Asterisk_Dollar 3.429291258943601 ns/iter 3.424748574474079 ns/iter 1.00
Regex_Caret_Group_Period_Asterisk_Group_Dollar 3.421590819879487 ns/iter 3.4194380630287022 ns/iter 1.00
Regex_Caret_X_Hyphen 13.689679118941607 ns/iter 13.242160032591093 ns/iter 1.03
Regex_Period_Md_Dollar 88.25481676213872 ns/iter 88.46860380323241 ns/iter 1.00
Regex_Caret_Slash_Period_Asterisk 8.401277030046996 ns/iter 8.0802780950559 ns/iter 1.04
Regex_Caret_Period_Range_Dollar 4.664719963799649 ns/iter 3.420101712911093 ns/iter 1.36
Regex_Nested_Backtrack 836.4587272221149 ns/iter 824.6595964961987 ns/iter 1.01

This comment was automatically generated by workflow using github-action-benchmark.

@jviotti jviotti force-pushed the clang-format-preinstall branch 2 times, most recently from 3d232b5 to 10158ac Compare June 20, 2025 15:10
@jviotti jviotti changed the title [WIP] Attempt to install ClangFormat binaries using CMake Attempt to install ClangFormat binaries using CMake Jun 20, 2025
@jviotti jviotti marked this pull request as ready for review June 20, 2025 15:10
@jviotti
Copy link
Member Author

jviotti commented Jun 20, 2025

@bavulapati The problem is that different OSes have different versions of clang-format in the package manager, that leads to formatting incompatibilities. That's why we currently pull clang-format using pipx on CI.

The way the new function works is that it will try to fetch a pre-built binary, else still fallback to the system one (if any). But for common platforms like macOS, you should always get a pre-built binary hit and then you can forget about version incompatibilities

@jviotti jviotti force-pushed the clang-format-preinstall branch from 10158ac to 2f41805 Compare June 20, 2025 15:12
@bavulapati
Copy link
Contributor

@bavulapati The problem is that different OSes have different versions of clang-format in the package manager, that leads to formatting incompatibilities. That's why we currently pull clang-format using pipx on CI.

The way the new function works is that it will try to fetch a pre-built binary, else still fallback to the system one (if any). But for common platforms like macOS, you should always get a pre-built binary hit and then you can forget about version incompatibilities

That makes sense 👍🏾

@jviotti jviotti force-pushed the clang-format-preinstall branch from 2f41805 to 660f3bb Compare June 20, 2025 15:24
@jviotti
Copy link
Member Author

jviotti commented Jun 20, 2025

I think additionally this means I will be able to get formatting work on the MSYS2 build, which I was struggling to make it work before. Maybe as a subsequent PR

@jviotti jviotti merged commit 74fdf81 into main Jun 20, 2025
14 checks passed
@jviotti jviotti deleted the clang-format-preinstall branch June 20, 2025 15:35
@bavulapati
Copy link
Contributor

@jviotti I frequently do make clean and this results in deleting ClangFormat binary too.
It would be better if we separate the build files and downloaded files.
What do you think about having a different folder for CLangFormat binaries?

@jviotti
Copy link
Member Author

jviotti commented Jun 24, 2025

Why do you run make clean? I found that to be very uncommon on a daily basis. CMake is smart enough to wipe things out and reconfigure when needed.

@bavulapati
Copy link
Contributor

I do that so clang-tidy config is in effect completely.
Just to be sure.

bavulapati added a commit to bavulapati/core that referenced this pull request Jun 26, 2025
Refs sourcemeta#1754
Inspired by sourcemeta#1739
This makes it use consistent version of clang-tidy on all the platforms.
And also uses fairy latest version of clang-tidy, instead of using old
versions shipped along with apple llvm and similar distros.

Signed-off-by: Balakrishna Avulapati <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants