sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: rust
Issue: Missed `match` optimization of riscv
Labels: A-LLVM,I-slow,A-codegen,C-bug,needs-triage

<!--
Thank you for filing a bug report! 🐛 Please provide a short summary of the bug,
along with any information you feel relevant to replicating the bug.
-->

I was code golfing different implementations for a function that "inverts" (rotates by 2) an enum with discriminants `1`, `2`, `4`, `8` (i.e. `f(1) = 4`), the obvious implementation being this:

```rust
pub enum Side {
    Top = 1 << 0,
    Right = 1 << 1,
    Bottom = 1 << 2,
    Left = 1 << 3,
}

pub fn opposite_match(x: Side) -> Side {
    use Side::*;
    match x {
        Top => Bottom,
        Right => Left,
        Bottom => Top,
        Left => Right,
    }
}
```

I expected to see this happen: on riscv `opposite_match` produces the optimal code, or at least a reasonable one.

Instead, this happened: compiler generates the following asm:
```asm
opposite_match:
        addi    a0, a0, -1
        slli    a0, a0, 24
        srai    a0, a0, 24
        lui     a1, %hi(.Lswitch.table.opposite_match)
        addi    a1, a1, %lo(.Lswitch.table.opposite_match)
        add     a0, a1, a0
        lbu     a0, 0(a0)

example::table_lookup::T::h07c70895e308d45b:
        .ascii  "\004\b\004\001\004\004\004\002"
```

The problem is the `slli` (shift left logical immediate) and `srai` (shift right arithmetic immediate) instructions -- those together are noop, since `a0 <= 7` at that point and `7 << 24 < 1.rotate_right(1)`.

The underlying issue is that llvm at some point replaces indexing by `u8` with indexing by `u32`, inserting a `sext` in the process; `sext` is not getting optimized out afterwards and is later lowered to `ssli`+`srai`. LLVM issue: https://github.com/llvm/llvm-project/issues/124841.

If you write the lookup table by hand compiler generates better asm:

```rust
pub fn table_lookup(x: Side) -> Side {
    static T: [Side; 8] = [
        Side::Bottom, // <--
        Side::Left, // <--
        Side::Bottom,
        Side::Top, // <--
        Side::Bottom,
        Side::Bottom,
        Side::Bottom,
        Side::Right, // <--
    ];
    T[x as usize - 1]
}
```
```
example::table_lookup::h8d56712109e87652:
        lui     a1, %hi(example::table_lookup::T::h07c70895e308d45b)
        addi    a1, a1, %lo(example::table_lookup::T::h07c70895e308d45b)
        add     a0, a1, a0
        lbu     a0, -1(a0)
        ret

example::table_lookup::T::h07c70895e308d45b:
        .ascii  "\004\b\004\001\004\004\004\002"
```

There are no shifts and the `-1` is merged into the the `lbu` (load byte unsigned).

[Godbolt link](https://godbolt.org/z/vGjGT9oW1).

### Meta
<!--
If you're using the stable version of the compiler, you should also check if the
bug also exists in the beta or nightly versions.
-->

rustc version:
```
1.86.0-nightly (2025-01-27 2f348cb7ce4063fa4eb4)
```
Repository: puppeteer
Issue: [Bug]: Puppeteer produces untagged PDF starting from v.23.8.0
Labels: bug

### Minimal, reproducible example

```TypeScript
const browser = await puppeteer.launch({ headless: "shell" });
const page = await browser.newPage();
await page.setContent(html);
await page.pdf({ path: outputPath });
await browser.close();
```

### Background

We are doing 3rd party dependencies scheduled upgrade for our product and noticed the issue with PDF generation when upgrading the puppeteer library. The resulting PDF must be tagged by default, this is already for a while (I believe from Chrome 81). After trying different versions we narrowed down the puppeteer version 23.7.1, which is still producing tagged PDF, but the very next version 23.8.0 produces PDF which is not accessible. We tried through the current version 24.1.1 and all of them do not produce tagged PDF.

### Expectation

Must produce tagged PDF by default.

### Reality

Starting from version 23.8.0 doesn't produce tagged PDF.

### Puppeteer configuration file (if used)

```TypeScript

```

### Puppeteer version

23.8.0 through 24.1.1

### Node version

Node 20

### Package manager

npm

### Package manager version

9.6.7

### Operating system

Windows
Repository: vscode
Issue: Node.js fetch failing with mitm proxy and basic auth
Labels: bug,proxy

> @chrmarti Yes that appears to be spot on
> 
> <details>
> 
> <summary>GitHub Copilot Chat Diagnostics output</summary>
> 
> ## GitHub Copilot Chat
> 
> - Extension Version: 0.24.2025012802 (prod)
> - VS Code: vscode/1.97.0-insider
> - OS: Mac
> 
> ## Network
> 
> User Settings:
> ```json
>   "http.proxy": "http://localhost:9999",
>   "github.copilot.advanced.debug.useElectronFetcher": true,
>   "github.copilot.advanced.debug.useNodeFetcher": false,
>   "github.copilot.advanced.debug.useNodeFetchFetcher": true
> ```
> 
> Connecting to https://api.github.com:
> - DNS ipv4 Lookup: 140.82.114.5 (11 ms)
> - DNS ipv6 Lookup: ::ffff:140.82.114.5 (5 ms)
> - Proxy URL: http://localhost:9999 (0 ms)
> - Proxy Connection: 407 Proxy Authentication Required
> 	proxy-authenticate: Basic realm="mitmproxy" (34 ms)
> - Electron fetch (configured): HTTP 200 (82 ms)
> - Node.js https: HTTP 200 (80 ms)
> - Node.js fetch: Error (23 ms): fetch failed
>   Request was cancelled.
>     Proxy response (407) !== 200 when HTTP Tunneling
> - Helix fetch: Error (174 ms): Miscellaneous failure (see text): no credential for 1221457B-73E4-463F-9A4A-B270AD5C8FF3
> 
> Connecting to https://api.githubcopilot.com/_ping:
> - DNS ipv4 Lookup: 140.82.114.22 (27 ms)
> - DNS ipv6 Lookup: ::ffff:140.82.114.22 (2 ms)
> - Proxy URL: http://localhost:9999 (1 ms)
> - Proxy Connection: 407 Proxy Authentication Required
> 	proxy-authenticate: Basic realm="mitmproxy" (31 ms)
> - Electron fetch (configured): HTTP 200 (74 ms)
> - Node.js https: HTTP 200 (88 ms)
> - Node.js fetch: Error (24 ms): fetch failed
>   Request was cancelled.
>     Proxy response (407) !== 200 when HTTP Tunneling
> - Helix fetch: Error (12 ms): Miscellaneous failure (see text): no credential for 1221457B-73E4-463F-9A4A-B270AD5C8FF3 (negative cache)
> 
> ## Documentation
> 
> In corporate networks: [Troubleshooting firewall settings for GitHub Copilot](https://docs.github.com/en/copilot/troubleshooting-github-copilot/troubleshooting-firewall-settings-for-github-copilot).
> </details>
> 
> Running a proxy, but not in wsl or any vs code remote, so I'm not sure why electron wouldn't be enabled, per: https://github.com/devm33/vscode/blob/419205a8ff1252fc6a3a8e085b8645f2ab51a0ec/extensions/github-authentication/src/node/fetch.ts#L6-L12
> 
> The `github-authentication` extension runs in the standard extension host right? 

 _Originally posted by @devm33 in [#207867](https://github.com/microsoft/vscode/issues/207867#issuecomment-2619408420)_
Repository: pytorch
Issue: Use of broadcast_shapes() errors attempting to guard on symbolic nested int
Labels: triaged,module: nestedtensor

Repro:

```python
import torch

torch._dynamo.config.capture_dynamic_output_shape_ops = True
torch._dynamo.config.capture_scalar_outputs = True

nt = torch.nested.nested_tensor([
    torch.randn(2),
    torch.randn(3),
    torch.randn(4),
], layout=torch.jagged, device="cuda")

@torch.compile(fullgraph=True)
def f(t, mask):
    nt = torch.nested.masked_select(t, mask)
    return torch.where(nt > 0., torch.ones_like(nt), torch.zeros_like(nt))

t = torch.randn(3, 5)
mask = torch.randint(0, 2, t.shape, dtype=torch.bool)
output = f(t, mask)
```

Problem: `torch.nested.masked_select()` constructs an NJT in-graph, which invokes a special path for constructing a new symbolic nested int. Attempting to guard on this (due to usage of `where()` -> `broadcast_shapes()` -> `Ne(s1, 1)` guard on the symbolic nested int) gives this error:

```
...
AssertionError: s0 (could be from ['<ephemeral: intermediate_offsets_or_lengths>']) not in {s0: []}.  If this assert is failing, it could be due to the issue described in https://github.com/pytorch/pytorch/pull/90665
```

cc @cpuhrsch @bhosmer @drisspg @soulitzer @davidberard98 @YuqingJ
Repository: PowerToys
Issue: Explorer shortcut for adding/removing a directory from PATH
Labels: Needs-Triage

### Description of the new feature / enhancement

When in a directory in Explorer, right clicking on empty space should reveal an option for adding the current directory to PATH if it isn't there already. If the directory already exists in PATH, the option should instead be a button for removing the current directory from PATH.

### Scenario when this would be used?

When installing a new program (like Postgres), it's common to not have useful tools available in PowerShell because the proper directories aren't added to PATH. Having a button in Explorer to add/remove these directories would make dealing with this issue easier.

### Supporting information

_No response_
Repository: flutter
Issue: `Mac ruby` failing due to ActiveSupport issue
Labels: team-infra

### Type of Request

bug

### Infrastructure Environment

cocoon

### What is happening?

```
 /opt/s/w/ir/x/w/cocoon/cipd_packages/ruby/build/bin/darwin_ruby/lib/ruby/gems/3.1.0/gems/activesupport-7.0.8/lib/active_support/logger_thread_safe_level.rb:12:in `<module:LoggerThreadSafeLevel>': uninitialized constant ActiveSupport::LoggerThreadSafeLevel::Logger (NameError)
     Logger::Severity.constants.each do |severity|
```

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac%20ruby/1197/overview

Rails tracking issue: https://github.com/rails/rails/issues/54260

I'm assuming, we need to pin the version of `concurrent-ruby` to `1.3.4` in https://github.com/flutter/cocoon/blob/main/cipd_packages/ruby/third_party/ruby_ship/ruby_ship_build.sh#L246

### Steps to reproduce

Run test

### Expected results

Expect test to pass
Repository: flutter
Issue: [SwiftPM] Remove warnings exception for google_sign_in_ios plugin
Labels: platform-ios,platform-mac,p: google_sign_in,P3,a: plugins,team-ios,triaged-ios

### Background

Xcode produces warnings for the AppAuth-iOS Swift package (see https://github.com/openid/AppAuth-iOS/issues/865), which is a dependency of the google_sign_in_ios plugin.

We cannot suppress these warnings when using Swift Package Manager. As a result, we added to the google_sign_in_ios plugin to the Xcode warnings exception list (see https://github.com/flutter/flutter/issues/146904).

### Work

Once the AppAuth-iOS warnings are addressed (see https://github.com/openid/AppAuth-iOS/pull/888), remove google_sign_in_ios from this exception list.
Repository: vscode
Issue: Disable telemetry reporting via GPO
Labels: triage-needed

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->
You asked for suggestions what Enterprise needs are - regarding future GPO settings.

1. [disable telemetry reporting](https://code.visualstudio.com/docs/supporting/FAQ#_how-to-disable-telemetry-reporting) and other privacy related settings.
Repository: flutter
Issue: `Mac ruby` failing due to missing command line tools
Labels: team-infra

### Type of Request

bug

### Infrastructure Environment

cocoon

### What is happening?

```
 Error: Xcode alone is not sufficient on Sonoma.
 Install the Command Line Tools:
   xcode-select --install
```

https://ci.chromium.org/ui/p/flutter/builders/try/Mac_arm64%20ruby/45/infra

### Steps to reproduce

Run test

### Expected results

Expect test to pass
Repository: TypeScript
Issue: Missing computed properties in declaration emit
Labels: Suggestion,In Discussion

With #60052, we now emit qualified names rather than generate an index signature for computed names in classes; however, this now puts us in an awkward place where regardless of errors, declaration emit diverges on `isolatedDeclarations`.

Playground Links:

* [5.7, without `isolatedDeclarations`](https://www.typescriptlang.org/play/?isolatedDeclarations=false&ts=5.7.3#code/KYDwDg9gTgLgBAYwgOwM7wOZWMGBLZDALjgFdk8BHU4OVATwFsAjCAGzgF44BlJ1tgAoAlAG4AUKEixEKdLPIwS5KjTr92XXhqFjxk8NHgI2AQ1So4AWXoBhMxYCMcAN7i4HuAG0sOfIQBdEnQoAgwtACIAd2g2ABMIiU9vJEUguGRSFmAoLQAWACYJAF99KSM4YEzGOAAFKAgwHJh6ADlTRmBLN2TfXDDIvv8MCIAad09U5HhuCKmYMfFSg2ljB0sbe3NUAtcJjy96xua2jq6AOiGw9JCB2ZioeMT97yOm2FPO1HP59OrmHL5IpLMqGGRsXBwMANMAAQmCMFChAkKwqJm21js6wAzHtkl5oY10iIuAA+OAANwgeDiWhJnHJLlKpSAA)
* [Nightly, without `isolatedDeclarations`](https://www.typescriptlang.org/play/?isolatedDeclarations=false&ts=5.8.0-dev.20250128#code/KYDwDg9gTgLgBAYwgOwM7wOZWMGBLZDALjgFdk8BHU4OVATwFsAjCAGzgF44BlJ1tgAoAlAG4AUKEixEKdLPIwS5KjTr92XXhqFjxk8NHgI2AQ1So4AWXoBhMxYCMcAN7i4HuAG0sOfIQBdEnQoAgwtACIAd2g2ABMIiU9vJEUguGRSFmAoLQAWACYJAF99KSM4YEzGOAAFKAgwHJh6ADlTRmBLN2TfXDDIvv8MCIAad09U5HhuCKmYMfFSg2ljB0sbe3NUAtcJjy96xua2jq6AOiGw9JCB2ZioeMT97yOm2FPO1HP59OrmHL5IpLMqGGRsXBwMANMAAQmCMFChAkKwqJm21js6wAzHtkl5oY10iIuAA+OAANwgeDiWhJnHJLlKpSAA)
* [Nightly, with `isolatedDeclarations`](https://www.typescriptlang.org/play/?isolatedDeclarations=true&ts=5.8.0-dev.20250128#code/KYDwDg9gTgLgBAYwgOwM7wOZWMGBLZDALjgFdk8BHU4OVATwFsAjCAGzgF44BlJ1tgAoAlAG4AUKEixEKdLPIwS5KjTr92XXhqFjxk8NHgI2AQ1So4AWXoBhMxYCMcAN7i4HuAG0sOfIQBdEnQoAgwtACIAd2g2ABMIiU9vJEUguGRSFmAoLQAWACYJAF99KSM4YEzGOAAFKAgwHJh6ADlTRmBLN2TfXDDIvv8MCIAad09U5HhuCKmYMfFSg2ljB0sbe3NUAtcJjy96xua2jq6AOiGw9JCB2ZioeMT97yOm2FPO1HP59OrmHL5IpLMqGGRsXBwMANMAAQmCMFChAkKwqJm21js6wAzHtkl5oY10iIuAA+OAANwgeDiWhJnHJLlKpSAA)

```ts
export const greeting: unique symbol = Symbol();
export const count: unique symbol = Symbol();

export class MyClass1 {
    [greeting]: string = "world";
    [count]: number = 42;
}

export enum PropertyNames {
    greeting = "greeting",
    count = "count",
}

export class MyClass2 {
    [PropertyNames.greeting]: string = "world";
    [PropertyNames.count]: number = 42;
}

export let prop!: string;

export class MyClass3 {
    [prop]: () => void = () => {}
}
```

Compared to `--isolatedDeclarations false` The declaration emit lacks expected contents in `MyClass1` and `MyClass2`:

```diff lang=ts
  export declare const greeting: unique symbol;
  export declare const count: unique symbol;
  export declare class MyClass1 {
-     [greeting]: string;
-     [count]: number;
  }
  export declare enum PropertyNames {
      greeting = "greeting",
      count = "count"
  }
  export declare class MyClass2 {
-     [PropertyNames.greeting]: string;
-     [PropertyNames.count]: number;
  }
  export declare let prop: string;
  export declare class MyClass3 {
      [prop]: () => void;
  }
```

It is surprising that `prop` is emitted regardless of `--isolatedDeclarations`, but not the other contents.
Repository: go
Issue: os/exec: LookPath considers paths containing ":" to be absolute on windows
Labels: Security,OS-Windows

os/exec.LookPath considers all paths containing `:` to be absolute, as one use of the colon is to indicate drive letters. Unfortunately it is also used for "alternate data streams" on NTFS filesystems, which allow attaching additional data to a file, accessed using a special suffix.

This can result in LookPath returning a file (with a ADS) in the current directory unexpectedly.

This is a PUBLIC track security issue per our security policy.

Thanks to Juho Forsén for reporting this issue.
Repository: electron
Issue: BaseWindow.contentView.addChildView(WebContentsView) is not rendering a page properly
Labels: platform/windows,bug :beetle:,34-x-y

### Preflight Checklist

- [x] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/main/CONTRIBUTING.md) for this project.
- [x] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/main/CODE_OF_CONDUCT.md) that this project adheres to.
- [x] I have searched the [issue tracker](https://www.github.com/electron/electron/issues) for a bug report that matches the one I want to file, without success.

### Electron Version

34.0.1

### What operating system(s) are you using?

Windows

### Operating System Version

Version 10.0.19045 Build 19045	

### What arch are you using?

x64

### Last Known Working Electron version

_No response_

### Expected Behavior

This is an excerpt of my entry file

```js
import { app, screen, BaseWindow, WebContentsView } from 'electron';

app.whenReady().then(() => {

    const 
        { workAreaSize } = screen.getPrimaryDisplay()
        ,
        parentView = new BaseWindow({
            width: workAreaSize.width,
            height: workAreaSize.height,
        })
        ,
        mainPage = new WebContentsView()
        ;

    if (parentView) parentView.maximize();

    parentView.contentView.addChildView(mainPage); // [FAILING] # failing even if commented out avoiding any clash of instruction set, only the line below works:..
    parentView.contentView = mainPage; // [PASSING] # temporary workaround (read "Additional Information" in this issue) 

    mainPage.webContents.loadFile('index.html');
    mainPage.webContents.openDevTools();

    parentView.on('closed', () => {
        mainPage.webContents.close();
    });

})
```

This is what I have expected to see:..

![Image](https://github.com/user-attachments/assets/64197452-e4ae-4feb-877d-becd0423ae27)


### Actual Behavior

This is what I truly get, i.e. only web page visible in Debugger's outliner, but not actually rendered in Web Contents' View:

![Image](https://github.com/user-attachments/assets/34026b2f-1760-4c54-9544-f10eb7ef6cb1)

### Testcase Gist URL

_No response_

### Additional Information

Possibly related to [the issue](https://github.com/electron/electron/issues/43255) recently raised. However it underlines a possible, although a temporary, workaround (see diff below): the drawback of the workaround itself is that it potentially overrides `BaseWindow.contentView.children` list, which is no-brainer BAD PRACTICE, especially if we need our list to contain multiple `View` instances !

```diff
// workaround (BAD PRACTICE)
- parentView.contentView.addChildView(mainPage); // [FAILING]
+  parentView.contentView = mainPage; // [PASSING]
```

If this will be accepted as bug by Electron's maintainers team, it means multiple View composition is and will be not possible until potential bug is fixed.

Thank you.
Repository: iptv
Issue: Add: Telecinco
Labels: streams:add

### Channel ID (required)

Telecinco.es

### Stream URL (required)

https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge3/smil:8_HD.smil/manifest.m3u8

### Quality

1080p

### Label

None

### Timeshift

_No response_

### HTTP User Agent

_No response_

### HTTP Referrer

_No response_

### Notes

_No response_

### Contributing Guide

- [x] I have read [Contributing Guide](https://github.com/iptv-org/iptv/blob/master/CONTRIBUTING.md)
Repository: go
Issue: cmd/go: toolchain directive can point to file relative to go.mod with ADS on windows
Labels: Security,OS-Windows

Due to https://github.com/golang/go/issues/71469, a toolchain directive with a ADS suffix (e.g. `toolchain go1.25-:alt`), can result in the toolchain attempting to execute a files alternate data stream that is located alongside the go.mod file.

Since it's somewhat complex to create a file with ADS, and as far as we could tell no source control software supports it, while clearly unexpected, we do not consider this significantly dangerous.

This is a PUBLIC track security issue per our security policy.

Thanks to Juho Forsén for reporting this issue.
Repository: iptv
Issue: Add: Antena3
Labels: streams:add

### Channel ID (required)

Antena3.es

### Stream URL (required)

https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge3/smil:7_HD.smil/manifest.m3u8

### Quality

1080p

### Label

None

### Timeshift

_No response_

### HTTP User Agent

_No response_

### HTTP Referrer

_No response_

### Notes

_No response_

### Contributing Guide

- [x] I have read [Contributing Guide](https://github.com/iptv-org/iptv/blob/master/CONTRIBUTING.md)
Repository: iptv
Issue: Broken: es.m3u LaSexta.es Boing.es Cuatro.es
Labels: broken stream

### Broken Links

[LaSexta.es](https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge2/smil:11_HD.smil/index.m3u8)
[Boing.es](https://spa-ha-p002.cdn.masmediatv.es/SVoriginOperatorEdge/smil:17_HD.smil/index.m3u8)
[Cuatro.es](https://spa-ha-p002.cdn.masmediatv.es/SVoriginOperatorEdge/smil:9_HD.smil/index.m3u8)

### What happened to the stream?

Not loading

### Notes (optional)

New links:
[LaSexta.es](https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge3/smil:11_HD.smil/manifest.m3u8)
[Boing.es](https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge/smil:17_HD.smil/index.m3u8)
[Cuatro.es](https://spa-ha-p005.cdn.masmediatv.es/SVoriginOperatorEdge3/smil:9_HD.smil/index.m3u8)
resolution does not change
### Contributing Guide

- [x] I have read [Contributing Guide](https://github.com/iptv-org/iptv/blob/master/CONTRIBUTING.md)
Repository: kubernetes
Issue: apiserver incompatible with cel-go 0.23.0
Labels: sig/api-machinery,needs-triage

I hit this error when a robot updated google/cel-go to 0.23.0 in a project that vendors k8s.io/apiserver v0.32.1:
```
# k8s.io/apiserver/pkg/cel/environment
vendor/k8s.io/apiserver/pkg/cel/environment/base.go:176:19: cannot use ext.TwoVarComprehensions (value of type func(options ...ext.TwoVarComprehensionsOption) "github.com/google/cel-go/cel".EnvOption) as func() "github.com/google/cel-go/cel".EnvOption value in argument to UnversionedLib
```
https://github.com/cilium/tetragon/pull/3290

I traced the problem, and it seems the incompatibility was introduced by https://github.com/google/cel-go/pull/1075/ that changed the type of `TwoVarComprehensions` (from `func() cel.EnvOption` to `func(...TwoVarComprehensionsOption) cel.EnvOption`).