[clang-offload-bundler] unification with the upstream LLVM [NFC]

[sndmitriev]

Unify FileHandler interfaces with the upstream LLVM.

Signed-off-by: Sergey Dmitriev [email protected]

[Fznamznon]

I believe we need a separate team of codeowners for clang-offload-bundler and clang-offload-wrapper changes.

[kbobrovs]

Unify FileHandler interfaces with the upstream LLVM.

@sndmitriev , could you please provide a link to the upstream with corresponding patch or provide some comments what this patch does?

[sndmitriev]

@kbobrovs, this is just a cleanup which does not change functionality. It makes FileHandler interface the same as in the llvm.org.

[bader]

@sndmitriev, is there any reason to not contribute this patch to llorg? If not, I suggest committing to llorg instead. We regularly pull commits from llorg, so if patch lands there, it will appear eventually in sycl branch.

[sndmitriev]

Do I need to do any additional changes to get this patch merged?

[bader]

Do I need to do any additional changes to get this patch merged?

Could you answer the question below, please?

@sndmitriev, is there any reason to not contribute this patch to llorg? If not, I suggest committing to llorg instead. We regularly pull commits from llorg, so if patch lands there, it will appear eventually in sycl branch.

[sndmitriev]

Do I need to do any additional changes to get this patch merged?

Could you answer the question below, please?

@sndmitriev, is there any reason to not contribute this patch to llorg? If not, I suggest committing to llorg instead. We regularly pull commits from llorg, so if patch lands there, it will appear eventually in sycl branch.

Sure. This patch removes differences in FileHandler interface between llorg and sycl branch that were introduced to sycl branch long time ago. FileHandler interfaces that I am removing in this patch just do not exist in llorg, so it cannot be committed to llorg.

[bader]

This patch removes differences in FileHandler interface between llorg and sycl branch that were introduced to sycl branch long time ago. FileHandler interfaces that I am removing in this patch just do not exist in llorg, so it cannot be committed to llorg.

I remember comparing llorg and sycl branches before posting my question, but it looks like I did something wrong, because I didn't that difference and I can see it. Sorry for the delay.