Aaron Patterson
Aaron Patterson
> [@tenderlove](https://github.com/tenderlove) seems not effective. Same exception. Could be related to fact jruby on windows consider "\n" as newline (same as CRuby). Ah, great. I didn't know it was the...
CI seems to be broken, but for JRuby only. Apparently [`Tempfile` doesn't have a create method](https://github.com/ruby/rubygems/actions/runs/19938329596/job/57169271283?pr=9154#step:22:371). Why is this different on JRuby? cc @headius
> Makes sense. Please add a test that previously didn't optimize, though. Do we not have tests for that already?
Ah, nevermind. I think understand. We need a test with `add reg, mem, mem` or something?
> This possibly admits more cases. Previously when `right` was an immediate it did not attempt to merge. So would be nice to have a test for those situations. I...
Sorry, I'm still not following 😢 ``` Sub ZeroReg(regno=31), 0xf => out Mov c, out ``` Is this not the same as: ``` Sub out, ZeroReg(regno=31), 0xf Mov c, out...
Never mind, I get it. `Sub ZeroReg(regno=31), 0xf => out` is LIR. `sub` with an immediate on arm64 can't take the zero register (it would be interpreted as SP rather...
> Is there a reason the uninstaller could't check the user gem path if the normal gem path does not exist? I don't think so. This seems like just a...
I think this makes sense. It's kind of unfortunate, but [this function](https://github.com/ruby/rubygems/blob/da0793a5b92e6cda5ded3cf0956f791cb042f81f/bundler/lib/bundler/source/rubygems.rb#L165) tightly couples [download](https://github.com/ruby/rubygems/blob/da0793a5b92e6cda5ded3cf0956f791cb042f81f/bundler/lib/bundler/source/rubygems.rb#L171) and [installation](https://github.com/ruby/rubygems/blob/da0793a5b92e6cda5ded3cf0956f791cb042f81f/bundler/lib/bundler/source/rubygems.rb#L181-L191). I would like to separate those two things at some point so that...