[PATCH] D11393: [X86] Allow X86::COND_NE_OR_P and X86::COND_NP_OR_E to be reversed.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 20:30:03 PST 2016


chandlerc added a subscriber: chandlerc.
chandlerc added a comment.

In http://reviews.llvm.org/D11393#334183, @davidxl wrote:

> LGTM -- watch out for test failures.


David, Gerolf suggested getting one of the x86 maintainers to look at this, and Nadav was CC-ed, but no one with deep knowledge of the x86 backend ever really commented on the patch. =/ Seems somewhat bad form to LGTM without getting one of the long standing maintainers to chime in here.

(And there does appear to be a problem with it, see the comment from James on the commit thread...)


Repository:
  rL LLVM

http://reviews.llvm.org/D11393





More information about the llvm-commits mailing list