[PATCH] D81230: Correctly report modified status for LoopIdiomRecognize

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 01:34:55 PDT 2020


foad added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:1067
 
+  ExpandedValuesCleaner EVC{Expander, TLI};
+
----------------
How does this work? I thought brace initialization was only for structs with public fields (but I'm clearly not an expert). And even if it does work, wouldn't parentheses be a bit more "normal" than braces?


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

https://reviews.llvm.org/D81230





More information about the llvm-commits mailing list