[PATCH] D42205: [X86] Add intrinsic support for the RDPID instruction

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 14:22:41 PST 2018


craig.topper added inline comments.


================
Comment at: lib/Support/Host.cpp:1260
+
+  Features["ibt"] = HasLeaf7 && ((EDX >> 20) & 1);
 
----------------
RKSimon wrote:
> Revert spacing change to keep = aligned.
I unaligned it because its checking EDX and the lines above are checking ECX. And the EBX lines above that have a different alignment.

While not paying attention previously I accidentally put the ibt line with the ECX lines as you can see in the before here. I was trying to discourage accidentally doing that again.


https://reviews.llvm.org/D42205





More information about the llvm-commits mailing list