[PATCH] D46801: add getAA in ScheduleDAG for using in HazardRec

Andrew Trick via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 15 10:09:05 PDT 2018


atrick added a reviewer: hfinkel.
atrick resigned from this revision.
atrick added a comment.
This revision now requires review to proceed.

Atheel,

Just to be clear, you aren't actually using PostRASchedulerList are you?

Once you have contributed a couple sizable patches, you can email Chris Lattner to get commit access.

I could commit this for you, but I just noticed something strange going on. There are two pointers to the current AliasAnalysis, one in ScheduleDAGInstrs and one in ScheduleDAGMI. It looks like they may be set under different conditions. I'm afraid combining them under the same getAA() virtual method may just create more confusion. It looks like Hal was last to work on this.

Hal, can you take a look and commit this in whatever form makes sense to you?


Repository:
  rL LLVM

https://reviews.llvm.org/D46801





More information about the llvm-commits mailing list