[PATCH] D49994: Allow constraining virtual register's class within reason
Alexey Zhikhartsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 31 05:45:25 PDT 2018
alexey.zhikhar added a comment.
In https://reviews.llvm.org/D49994#1181808, @jonpa wrote:
> Is this making sense, and what are your thoughts? What happens if you run your out-of-tree test cases with the SystemZ (-mcpu=z13) backend? Do you see any improvement? In that case, it seems like we have a test of real worth...
The test cases that we have heavily rely on our backend's intrinsics, so they cannot be compiled for SystemZ.
Providing a reliable test case for this patch does not seem as an easy (or even worthwhile) task but I'm open to suggestions.
Repository:
rL LLVM
https://reviews.llvm.org/D49994
More information about the llvm-commits
mailing list