[PATCH] D45146: [x86] Introduce a pass to begin more systematically fixing PR36028 and similar issues.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc marked 2 inline comments as done.
chandlerc added a comment.

In https://reviews.llvm.org/D45146#1054076, @craig.topper wrote:

> I may go ahead and pull the X86InstrArithmetic.td changes out of this. The SchedRW is incomplete for ADCX, it needs a ReadAfterLd.


After talking on IRC, I've just landed this part of the patch. It's less broken and should make it easy for you to improve this stuff.

> Shoudl we pull out the X86InstrInfo.cpp changes too? Is this the same as the change in the spectre load hardening patch too?

Done.

Also, addressed the minor comments inline.


Repository:
  rL LLVM

https://reviews.llvm.org/D45146