[PATCH] D79037: [StructurizeCFG] Fix region nodes ordering

Sameer Sahasrabuddhe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 31 21:50:12 PDT 2020


sameerds accepted this revision.
sameerds added a comment.
This revision is now accepted and ready to land.

This looks really good. Thanks a lot for bringing it this far! Please do check the #include for LoopInfo.h before submitting.



================
Comment at: llvm/lib/Transforms/Scalar/StructurizeCFG.cpp:17
 #include "llvm/Analysis/LegacyDivergenceAnalysis.h"
 #include "llvm/Analysis/LoopInfo.h"
 #include "llvm/Analysis/RegionInfo.h"
----------------
I think this can be removed too. At least a quick grep did not show any use of Loop or LoopInfo other than the code that gets eleminated with this patch.


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

https://reviews.llvm.org/D79037





More information about the llvm-commits mailing list