[PATCH] D34555: [NVPTX] Add lowering of i128 params.

[Artem Belevich via Phabricator via llvm-commits]

tra requested changes to this revision.
tra added a comment.
This revision now requires changes to proceed.

In https://reviews.llvm.org/D34555#802008, @denzp wrote:

> Is it safe just to change data layout at clang side? Won't it break backward compatibility?


Clang and LLVM sides of this change should land together as they have to be in sync.

Other than that, AFAICT it should be safe as it affects only i128 which is not currently supported, so there should be no backward compatibility issues. That's in theory. You may want to ask this question on cfe-dev at .


Repository:
  rL LLVM

https://reviews.llvm.org/D34555