leonardchan added subscribers: fedor.sergeev, philip.pfaffe, leonardchan.
leonardchan added a comment.

cc: @fedor.sergeev @philip.pfaffe on this

I think one of the things we want in the new PM is the data locality that we 
see from iterating only specific IR units (functions in this case). This was 
brought up before when porting ASan (D54337 <https://reviews.llvm.org/D54337>). 
In this patch it also looks like the pass still goes over functions only. If 
there are other globals that this should pass over, could you add/update a test 
to show that?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64843/new/

https://reviews.llvm.org/D64843



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to