[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