@thekbknapp I decreased full release build times for ripgrep by a factor of 2 (68s -> 33s) with a single change: I removed the vec-map feature on clap.
-
Show this thread
-
This was a completely unintentional change. I was just trying to trim some dependency fat and saw this as a candidate. I couldn't observe any runtime performance difference, but was amazed at compile time decrease!
1 reply 0 retweets 3 likesShow this thread -
Replying to @burntsushi5
I did some (unscientific) tests, and I think this may be due to the jump from a version prior to 2.29.2 to using 2.29.2. Reason being 2.29.2 is the version that I pretty massively debloated (from an compile codegen pov).
1 reply 0 retweets 0 likes -
Replying to @thekbknapp
Hmm but in my comparison I was using the same version of clap? Or am I misunderstanding? (Or did I get something wrong?)
1 reply 0 retweets 0 likes -
Replying to @burntsushi5
The plot thickens! I'm not sure :/ I tested both w/wo vec_map and couldn't get any meaningful change to build times. Then I tried to do the same with 2.29.2 I *also* couldn't see a diff w/wo vec_map. But then I noticed the diff of 2.26.2 to 2.29.2 was 59s->38s.
1 reply 0 retweets 0 likes
Hmmmmmmmmm... :thinking_face:
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.