[PATCH] D41721: [X86] Move HasNOPL to a subtarget feature bit. Plumb MCSubtargetInfo through the MCAsmBackend constructor

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 7 02:28:55 PST 2018


RKSimon added a comment.

In https://reviews.llvm.org/D41721#969305, @craig.topper wrote:

> Should I go back to my first patch that just does the plumbing for now?


Yes please, and split off the NOP length changes to a seperate patch to allow for regression tests. That way I think this patch should have no effect on codegen and is just a cleanup?


https://reviews.llvm.org/D41721





More information about the llvm-commits mailing list