perry-ca wrote:
> I just happened to look into the implementation of AutoConvert.h, and I see
> the entire interface is defined away when not targeting zOS. This means that
> all call sites need to be conditional on `ifdef __MVS__`, which means we have
> ugly ifdef droppings all over the codeb
https://github.com/rnk closed https://github.com/llvm/llvm-project/pull/135430
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rnk wrote:
I just happened to look into the implementation of AutoConvert.h, and I see the
entire interface is defined away when not targeting zOS. This means that all
call sites need to be conditional on `ifdef __MVS__`, which means we have ugly
ifdef droppings all over the codebase, which ha
https://github.com/zibi2 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/135430
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/abhina-sree approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/135430
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Sean Perry (perry-ca)
Changes
The commit
https://github.com/llvm/llvm-project/commit/a1935fd3809772c06f9a09fa151181642ae92b20
removed an include that is needed when building on z/OS.
---
Full diff: https://github.com/llvm/llvm-project/
https://github.com/perry-ca created
https://github.com/llvm/llvm-project/pull/135430
The commit
https://github.com/llvm/llvm-project/commit/a1935fd3809772c06f9a09fa151181642ae92b20
removed an include that is needed when building on z/OS.
>From 97b19d40e42752a980c5584e758570838def193a Mon Se
